Here is my first feedback for the Dialog for the Customer-managed egress.
- Wild-card doesn’t include main domain. For example I want to use .github.com. (for like api.github.com, git.gisthub.com). The ‘’ doesn’t include the main domain, from my initial testing. This then adds the extra listing / duplication on the approval dialog.
→ I would love to have a way to allow the main domain including sub-domain. Or have the wild-card allow the main domain as well.
- What is the difference between ‘Image’ and ‘Media’. Why this difference? This looks to me like ‘implementation’ details leaking into the UI. As a end user a ‘image’ and a ‘media’ is very, very similar. Both can show stuff. The main difference is (I assume), that media can include audio. And makes video more efficient.
→ Generally, as a admin I don’t care about this difference.
The other categories make sense.
- Links require the
Fetch client side permission? I’m not 100% sure yet. But if that is true, it feels a bit odd / clunky. Having ‘working’ links without any pop-up is something everyone is used to. Having that requiring ‘Fetch client side’ permission is very odd.
- The listing gets quickly very, very long, because each allowed type needs a row (see screenshot)
- The performance of the Dialog is awful. (At least for EU, but it seems more than just a rountrip issue). Its about 2 seconds! The network round-trip is way lower. It just feels broken / the app is hanging if you call it.
Small note: I though I maybe do something wrong and thats why the Dialog is so abysmally (looks broken) slow.
But the `Data management` in the Connected apps has similar issues. Especially the ‘Delete’ button reaction takes so long, that it feels broken. aka, the you start to click it again because you thing it didn’t work.
The dialog performance matters on how we’ll design our UI.
I basically intended to prompt for approval on the ‘Save’ button. But that gives a such a confusing ‘hanging app’ experience.
It basically would need some extra ‘immediate’ step, where we show indicate that soon some extra UI wills how up and then the app will continue.
Some additional feedback on Managed Egress:
- We’ve observed that
permissions.egress.set() appears to be a full-group replacement operation and re-triggers the admin approval dialog for the entire group, including domains that were previously approved and unchanged. A bit confusing to the user if they have previously approved entries.
- We also experimented with using
set() to remove entries instead of deleteDomain(), but the resulting group state did not always update consistently.
- When using the
deleteDomain API, it is not clear if the user needs to actually approve the deletion. A dialog does not open in those cases. I am ok with this but the docs should make it clear what the expected behavior is.
- The delete domain UI feels buggy as usually the first click does nothing and you have to click a second time for it to take effect and it is slow.
