Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kapua UI Freezes When Users Phone Number Too Long #944

Closed
LeoNerdoG opened this issue Nov 9, 2017 · 1 comment
Closed

Kapua UI Freezes When Users Phone Number Too Long #944

LeoNerdoG opened this issue Nov 9, 2017 · 1 comment
Assignees
Labels
Bug This is a bug or an unexpected behaviour. Fix it! Console GWT This issue/PR is related to Admin Web Console

Comments

@LeoNerdoG
Copy link
Contributor

LeoNerdoG commented Nov 9, 2017

When adding phone number to a user, Kapua does not return any warning or an error if number is too long. Instead it freezes and the whole UI has to be refreshed.

Testflow:

  1. Login as kapua-sys
  2. Go to Users, create new user (e.g. kapua-user1)
  3. Add phone number longer than 64 digits
    Kapua shows that is somehow processing the data, but never finishes (look at the screenshot).

screen shot 2017-11-09 at 09 29 12

Browsers used:

  • Google Chrome (Version 61.0.3163.100 (Official Build) (64-bit))
  • Safari (Version 11.0 (12604.1.38.1.7))
  • Mozilla Firefox (56.0.1 (64-bit))
  • Opera (Version 48.0.2685.39)
@LeoNerdoG LeoNerdoG changed the title No Error From Kapua When Phone Number Too Long Kapua UI Freezes When Phone Number Too Long Nov 9, 2017
@LeoNerdoG LeoNerdoG changed the title Kapua UI Freezes When Phone Number Too Long Kapua UI Freezes When Users Phone Number Too Long Nov 9, 2017
@lorthirk lorthirk added the Console GWT This issue/PR is related to Admin Web Console label Nov 14, 2017
@lorthirk lorthirk added the Bug This is a bug or an unexpected behaviour. Fix it! label Nov 15, 2017
bojanutkovic added a commit to bojanutkovic/kapua that referenced this issue Nov 15, 2017
Signed-off-by: Bojan Utkovic <bojan.utkovic@comtrade.com>
@LeoNerdoG
Copy link
Contributor Author

I have verified that this is resolved with PR #973 .

lorthirk pushed a commit that referenced this issue Nov 15, 2017
* user now can not add job with same name

Signed-off-by: Bojan Utkovic <bojan.utkovic@comtrade.com>

* PR Cleanup

Signed-off-by: Claudio Mezzasalma <claudio.mezzasalma@eurotech.com>

* #944 #945 #946

Signed-off-by: Bojan Utkovic <bojan.utkovic@comtrade.com>

* PR Cleanup

Signed-off-by: Claudio Mezzasalma <claudio.mezzasalma@eurotech.com>

* Length validation on UserAddDialog fields

Signed-off-by: Claudio Mezzasalma <claudio.mezzasalma@eurotech.com>
lorthirk pushed a commit that referenced this issue Nov 15, 2017
* user now can not add job with same name

Signed-off-by: Bojan Utkovic <bojan.utkovic@comtrade.com>

* PR Cleanup

Signed-off-by: Claudio Mezzasalma <claudio.mezzasalma@eurotech.com>

* #944 #945 #946

Signed-off-by: Bojan Utkovic <bojan.utkovic@comtrade.com>

* PR Cleanup

Signed-off-by: Claudio Mezzasalma <claudio.mezzasalma@eurotech.com>

* Length validation on UserAddDialog fields

Signed-off-by: Claudio Mezzasalma <claudio.mezzasalma@eurotech.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a bug or an unexpected behaviour. Fix it! Console GWT This issue/PR is related to Admin Web Console
Projects
None yet
Development

No branches or pull requests

3 participants