Hey Tadeas,as agreed on the IRC, I have opened these jiras:https://issues.jboss.org/browse/AGPUSH-1423 UPS Console: bulk of Admin UI/UX issueshttps://issues.jboss.org/browse/AGPUSH-1421 UPS Console: re-introduce double-check confirmation for app/variant remove/secret-resethttps://issues.jboss.org/browse/AGPUSH-1422 Describe test spec for testing new Push Opened Analytics featureCheers,~ LukasOn Tue, May 26, 2015 at 10:40 AM, Matthias Wessendorf <matzew@apache.org> wrote:Hi,thanks for the testing. Looks like we are good to go w/ the beta.1 release.Like Lukas already asked, are you able to create a few JIRAs to capture the issues on the UI ?ThanksOn Fri, May 22, 2015 at 1:30 PM, Lukáš Fryč <lukas.fryc@gmail.com> wrote:Thanks Tadeas,that's wonderful feedback!On Fri, May 22, 2015 at 3:24 AM, Tadeas Kriz <tkriz@redhat.com> wrote:UnifiedPush Server 1.1.0-beta.1 testing report
Hello there again,
so I have tested the UPS using the UI console, iOS 8.0 and Android 5.1. I have used the HelloPush example as to me it seems the best for testing the server and messaging.
I have not tested the Windows, SimplePush and Amazon. The Simple Push was tested in the automated integration tests, but that does not apply for Windows and Amazon as we do not have the test cases for those. I also could not test them manually because I do not have Windows nor Amazon developer accounts.
I have tortured the Admin UI quite thoroughly and it did really well. There were some bugs and some UX quirks that I suggest we fix into the next beta release.
Kudos for:
- Absolutely f***ing awesome wizard in the UnifiedPush Server Admin UI. I have loved every step of it and what totally amazed me was that when I deployed the app on the device and the app registered itself to the UPS, the wizard automagically progreessed to the next step! That is so damn awesome and the UX is just wonderful.
- It was so great that I (as a user) would not mind if it was shown also when creating another variants in an application that already has some, because it makes it more friendly and gives the feedback when the first registration is successful.
- It is also great that the user does not need to click on a link to keep logged in and only focusing the window is enough. Thanks for that.
- iOS HelloPush example is Swift 1.2 ready!
- Very nice error reporting in iOS HelloPush example (I set the URL wrong and the app told me about registration issue)
Found bugs
When sending push message using the UI, the following can happen:
- Check one or more variants in the variants list
- Uncheck all of the variants (this will leave you in the same visual state as it was before step 1.)
- Send the message
- Wonder why it was not sent to any of the devices
It seems that when I open the dialog, it shows
No variants
, but it really meansAll variants
. However when I check some variants and then uncheck them all, it will still sayNo variants
, but now it really means it asNo variants
. My suggestion is to start with having all the variants checked and instead of listing them all separated by commas, we should showAll variants
.For some strange reason there is a lot of space in the bottom and I can scroll away all the content.
- There are
?
help icons/buttons in the lower-right corners [5] in the analytics panel, but they do not do or show anything.- In the variant list it still shows
0 delivered
[7] even when I already sent and delivered messages.- In
Sender API
tab there is alwaysSet up Java UPS
on top of the other sender platform (see [2, 3, 4])- Clicking the
Read more
about master secret and push app id security [8] leads to/ag-push/#
(and somehow corrupts the history and I could not hit back button).- Search in PushApplication list does not work (and I am not sure if it is necessary as there should not be that many apps).
UI/UX Quirks
- Variant creation dialog wraps text when there is a lot of space [1].
- The area between number of devices and
edit
button shows thehand
cursor, but it is not clickable. I would suggest to make it clickable and the action would be expanding/collapsing the variant detail.- The
+
/-
button for expanding/collapsing the statistics does not showhand
cursor when mouse is above it.- In the top right corner of the window, there is the
warnings
information, however it is strange when there are no warnings so I would suggest one of these:
- Do not show it in the bar altogether when there are no warnings
- Disable the
clickability
of it when there are no warnings- When the popup is shown, it should not be just an empty box[6], but should show something like
No warnings
- When hitting the
Edit
variant button, it shows a dialog where I can change only the name of the variant. However when I click theChange network options
button, I am presented with the possibility to change both the name of the variant and the network specific configuration. I would therefore suggest to not show the name field in theChange network options
dialog as it is not considered a good practise to have two ways of doing the same thing in UI.- It is a bit strange that when I hover over the
copy
(sources) in variant detail, theedit
anddelete
buttons disappear until I move the mouse away from thecopy
.- Links to
Android
,Chrome
,iOS
,Cordova
etc. in variant detail should open the documentation in a new tab like the rest of documentation links. Now it opens in the same app which is not good because the user wants to stay in the Admin UI.- When I click on the
in the documentation
link on theno variants
screen, I get redirected correcly, but it takes a while for the images to load and it pushes the content away and I lose track where was the help I came for. I would suggest to set the dimensions into the <img> tags therefore it will not push the screen away, but simply load into the empty space.This is aerogear.org bug, could you report that?
- Each time the user wants to do a destructive action (i.e.
deleting application
,deleting variant
,reseting master secret
) he should be presented with a dialog in which he would have to type the name of thevariant
orapplication
in order to continue. This was already in before and I am not sure why was it removed.We have discussed this with Andres and he believes it is unnecessary from UX PoV: clicking big red button should be enough. :-)But since this is re-occuring report to the new UI, we should probably rethink that and introduce confirmation back.Questions
- I am not sure what
x receivers / y opened
means as well as the meaning offirst time opened
andlast time opened
.- Is there a way to access the list of registered installations as it was before? Should it?
- In analytics there are
Push opens
statistics. Does that mean there is some new API in UnifiedPush Server? If so should we test it? If so how?Yes, all these are related to the new Analytics - push notification sent to devices / devices opened metric.One more thing
It would be great if we implemented documentation versioning similar to readthedocs.org. That way there could be more versions of the documentation hosted on the aerogear.org site and each version of UnifiedPush Server Admin UI could point to the specific version. It would lead to better user experience because they would see the documentation exactly for their version.
Something what we could consider is hosting generated documentation in version-prefixed directories, such as here:This is probably worth separated thread.All taken into consideration the team did a great job on the 1.1.0 and even though it is still just beta.1, it is working very well. Thanks guys!
PS: Sorry for the order of images, it is 3 am and I was too tired to reorganize them. Also I did not yet create any JIRA tickets for the bugs. I just wanted to share the report with you guys so we can discuss which of the items will be moved into the JIRA as tickets and which are not important or would me marked as "won't fix".
@Tadeas: could I ask you to report the bugs in one or two JIRAs (+ JIRAs specific to aerogear.org, such as page scrolling issue)? Let's report them in a bulk and assigned to me, I will split them as necessary into subtasks.Thanks,
Tadeas Kriz1 - https://lh5.googleusercontent.com/-OHKxiXEIT88/VV6DvB0uRKI/AAAAAAAAX5I/a-m6umCfZUo/w593-h758-no/Screen%2BShot%2B2015-05-22%2Bat%2B12.14.05%2Bam.png
2 - https://lh5.googleusercontent.com/-KzruF4kzwj4/VV6Dyuu1kxI/AAAAAAAAX5o/RFuCFGzxtXo/w1192-h512-no/Screen%2BShot%2B2015-05-22%2Bat%2B1.27.53%2Bam.png
3 - https://lh6.googleusercontent.com/-ZAlJzi51BX0/VV6Dx1J3SxI/AAAAAAAAX5k/UQiql9TMye4/w1187-h571-no/Screen%2BShot%2B2015-05-22%2Bat%2B1.27.58%2Bam.png
4 - https://lh6.googleusercontent.com/-LHyvZPW5AzI/VV6DxSuf-yI/AAAAAAAAX5c/bi9eJaQ6RLc/w1199-h491-no/Screen%2BShot%2B2015-05-22%2Bat%2B1.28.04%2Bam.png
5 - https://lh6.googleusercontent.com/-A-dpKMM4IcU/VV6D1WjHp5I/AAAAAAAAX54/XeQC-2KUqVg/w373-h146-no/Screen%2BShot%2B2015-05-22%2Bat%2B1.12.15%2Bam.png
6 - https://lh3.googleusercontent.com/-dU4Weqwf5as/VV6D2OFf-II/AAAAAAAAX6A/T5CDLREzT-w/w277-h83-no/Screen%2BShot%2B2015-05-22%2Bat%2B1.09.51%2Bam.png
7 - https://lh5.googleusercontent.com/-q_0F21tJZgo/VV6Dz4jZJPI/AAAAAAAAX5w/ngGABCT_AUM/w320-h316-no/Screen%2BShot%2B2015-05-22%2Bat%2B1.19.53%2Bam.png
8 - https://lh5.googleusercontent.com/-57qlJTvJEBI/VV6Dwg3i0SI/AAAAAAAAX5Q/KPQxgw1DS0w/w644-h63-no/Screen%2BShot%2B2015-05-22%2Bat%2B1.28.56%2Bam.png--On Thu, May 21, 2015 at 11:39 PM, Tadeas Kriz <tkriz@redhat.com> wrote:Hey,I've just run integration tests and the UnifiedPush Server works great. One huge downside is the change that breaks the API of Java Sender and that the API is currently in somehow inconsistent state (it is synchronous, but it uses callback to deliver the `success` state and throws exception to deliver the `failed` state).Now I'm going to proceed with the real-device testing.On Thu, May 21, 2015 at 10:33 PM, Matthias Wessendorf <matzew@apache.org> wrote:_______________________________________________NEVERMIND, see https://github.com/aerogear/aerogear-unifiedpush-server/pull/568 for detailsstaging stays, and we continue w/ 1.1.0 of KeycloakOn Thu, May 21, 2015 at 10:18 PM, Matthias Wessendorf <matzew@apache.org> wrote:please give it a spin, so that I can merege it, in order to actually do the re-stagingOn Thu, May 21, 2015 at 10:15 PM, Matthias Wessendorf <mwessendorf@gmail.com> wrote:we need to re-stage, due to this bug:On Thu, May 21, 2015 at 9:21 PM, Matthias Wessendorf <mwessendorf@gmail.com> wrote:Hi,after month of work, here is the first beta release for the UPS 1.1.0. It contains more features and inprovements around UI, JMS for enhanced scalability and a lot of other stuff: https://issues.jboss.org/browse/AGPUSH/fixforversion/12326579Please test the staged release:Like w/ the previous alpha.2, please make sure you use a full profile WildFly or EAP server for tests, since we now have JMS hooks ;-)(See README for details)On Wednesday I'd like to press the button to release it to the wild.PS: Since this is the first beta release we won't yet be updating our Openshift cartridge - that will stay on 1.0.3 (stable) for a little longer time. For the next release (beta.2) in a few weeks we may get to this Openshift update.Thanks,Matthias--Matthias Wessendorf
blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf--Matthias Wessendorf
blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf--Matthias Wessendorf
blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf--Matthias Wessendorf
blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev----Tadeas Kriz--Tadeas Kriz
_______________________________________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev
_______________________________________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev--Matthias Wessendorf
blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf
_______________________________________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev
_______________________________________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev