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

Fixing Issue #578: Adding deletion dialog #889

Merged
merged 1 commit into from
Apr 23, 2018
Merged

Conversation

mhuang-ml
Copy link
Contributor

Adding an additional dialog message to the 2.0 development branch
that will require the user to press yes twice before the entity
and all associated flows and code are deleted

Adding an additional dialog message to the 2.0 development branch
that will require the user to press yes twice before the entity
and all associated flows and code are deleted
@mhuang-ml mhuang-ml requested review from grechaw and aebadirad April 16, 2018 21:59
Copy link
Contributor

@aebadirad aebadirad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Go ahead and implement this change in the develop branch as well, after consideration it's valuable till we can break entities out in the next release. Sorry for having you undo it the first place.

@aebadirad aebadirad merged commit d450fe3 into 2.x-develop Apr 23, 2018
aebadirad pushed a commit that referenced this pull request Apr 23, 2018
Adding an additional dialog message to the 2.0 development branch
that will require the user to press yes twice before the entity
and all associated flows and code are deleted
@aebadirad aebadirad deleted the issue/578 branch April 23, 2018 22:25
aebadirad added a commit that referenced this pull request Apr 24, 2018
* DHFPROD-496 fix QuickStart naming in app

“QuickStart”, not “Quick Start”

* DHFPROD-497 fix property index selection

Add flags to property to keep track of what indexes are set (instead of using name).
Set flags based on entity state when editor opens.
Set entity state based on flags on save.

* Fixes #582 - should be reviewed for UX before merging. (#906)

* Fix tab bar layout issue #798 (#909)

* Rework of fix for issue #557 where URI in request to /doc API (#915)

Now matches other implementations throughout codebase which leverage encodeURIComponent()

* Fixing Issue #578: Adding deletion dialog (#889) (#917)

Adding an additional dialog message to the 2.0 development branch
that will require the user to press yes twice before the entity
and all associated flows and code are deleted

* DHFPROD-664 adjust offset, size of new entities in UI (#918)
aebadirad added a commit that referenced this pull request Apr 24, 2018
* DHFPROD-496 fix QuickStart naming in app

“QuickStart”, not “Quick Start”

* DHFPROD-497 fix property index selection

Add flags to property to keep track of what indexes are set (instead of using name).
Set flags based on entity state when editor opens.
Set entity state based on flags on save.

* Fixes #582 - should be reviewed for UX before merging. (#906)

* Fix tab bar layout issue #798 (#909)

* Rework of fix for issue #557 where URI in request to /doc API (#915)

Now matches other implementations throughout codebase which leverage encodeURIComponent()

* Fixing Issue #578: Adding deletion dialog (#889) (#917)

Adding an additional dialog message to the 2.0 development branch
that will require the user to press yes twice before the entity
and all associated flows and code are deleted

* DHFPROD-664 adjust offset, size of new entities in UI (#918)

* Updating Issue #578: Update the deletion message (#919)

Removing the double confirmation dialog and updating the
deletion message that is displayed to the user to make it
clearer to what they are deleting.
aebadirad pushed a commit that referenced this pull request Oct 24, 2019
Adding an additional dialog message to the 2.0 development branch
that will require the user to press yes twice before the entity
and all associated flows and code are deleted
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants