Skip to content

Add delete confirmation to static pages. - #787

Merged
peter-fogg merged 1 commit into
masterfrom
peter-fogg/static-page-delete
Aug 23, 2013
Merged

Add delete confirmation to static pages.#787
peter-fogg merged 1 commit into
masterfrom
peter-fogg/static-page-delete

Conversation

@peter-fogg

Copy link
Copy Markdown
Contributor

While looking into the static pages I noticed that we were still using a JS confirm on deletion; this makes the UI consistent with the rest of Studio.
@singingwolfboy @talbs

We used to use a regular JS `confirm`; this makes the UI consistent
with the rest of Studio.
@cahrens

cahrens commented Aug 23, 2013

Copy link
Copy Markdown

👍

peter-fogg pushed a commit that referenced this pull request Aug 23, 2013
Add delete confirmation to static pages.
@peter-fogg
peter-fogg merged commit 5cb1b25 into master Aug 23, 2013
@peter-fogg
peter-fogg deleted the peter-fogg/static-page-delete branch August 23, 2013 18:52
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request May 9, 2016
…invitation

Fix bug of ContractRegister. openedx#786
Fix bug of access to Course About and Course Info. openedx#788
andrey-canon pushed a commit to eduNEXT/edx-platform that referenced this pull request Jun 21, 2018
DanielVZ96 referenced this pull request in open-craft/openedx-platform Jan 31, 2024
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