| Below my 2 cents why is there an asterisk beside the field name? ( App Name)* That's there to follow patternfly guidelines: https://www.patternfly.org/v3/pattern-library/forms-and-controls/field-labeling/#indicating-required-fields It seems a little redundant to have a menu with a single "delete" entry on these. I agree. That menu used to have an 'edit' option to allow application renaming, but such feature has been removed in this release.
I like the idea of the red icon to remove apps.
Not sure about the batch operation:
- do the user will have so many applications to delete to ever need a batch operation?
- giving the user the ability to delete a bunch of apps at once will make it easier to delete apps by mistake
The list of services on the left have a title and a subtitle I believe there has been a discussion about removing it. I was sure there was an issue about that but couldn't find it... The 3rd screen that temporarily appears in the dialog, saying "you can close this wizard" right before it closes itself before you finish reading what it says It closes because it gets automatically closed when the binding is done. I'm not a big fan of that either.
I would propose to remove the 'auto-close' and put a 'spinner' that is changed to a green check if the binding is successful or to a red cross if the binding fails. why does the one created by MDC have a suffix of "-android-ups-variant"? They have a suffix because for ONE app, we can have 2 variants: one for ios and one for android and they have to have different naming.
We could reduce it to just 'android' and 'ios' or something else. Or we could probably just use the CR name as you suggest. Why can I only create one variant of each type for Push? I'm not sure I remember the discussions behind this decision. One thing that comes in my mind is that if we create more than 1 variant for each platform (ios/android),
adding a `-android` and a `-ios` suffix to the app name won't suffice to make it unique. If we want to create more than 1 variant for platform, we will probably
need to add a UUID. |
