Skip to content
This repository has been archived by the owner on May 24, 2023. It is now read-only.

Handle enable set member failure #5

Closed
jnunemaker opened this issue Feb 18, 2013 · 0 comments · Fixed by #14
Closed

Handle enable set member failure #5

jnunemaker opened this issue Feb 18, 2013 · 0 comments · Fixed by #14
Labels
bug

Comments

@jnunemaker
Copy link
Owner

Right now if you add a member to a set (group, actor) and the ajax fails behind the scenes, no error is shown and the dom elements stays in the list.

Instead, we should show an error in the UI and also remove the failed enable from the set members.
gabceb added a commit to gabceb/flipper-ui that referenced this issue May 14, 2013
@gabceb
- Adds UI validation for percentage values fixes jnunemaker#7
c45f108 
- Errors alerts are shown an ajax requests fails with 404 or server failure fixes jnunemaker#6 fixes jnunemaker#5
@gabceb gabceb mentioned this issue May 14, 2013
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

UI validation and errors gabceb/flipper-ui
1 participant
@jnunemaker