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

Change delete asset prompt to Warning type for consistency. #432

Merged
merged 1 commit into from
Jul 22, 2013

Conversation

peter-fogg
Copy link
Contributor

No description provided.

@chrisndodge
Copy link
Contributor

+1 from me. @talbs can you also sign off so we can merge this?

@talbs
Copy link
Contributor

talbs commented Jul 22, 2013

Thanks for the work @peter-fogg. This looks solid to me.

👍

peter-fogg pushed a commit that referenced this pull request Jul 22, 2013
Change delete asset prompt to Warning type for consistency.
@peter-fogg peter-fogg merged commit 9a61038 into master Jul 22, 2013
@peter-fogg peter-fogg deleted the peter-fogg/update-asset-deletion-prompt branch July 22, 2013 13:10
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
Move mitxmako out of common lib, and pass template rendering function in...
antoviaque referenced this pull request in open-craft/edx-platform May 15, 2015
Consistently use display_name_with_default in Studio
diegomillan pushed a commit to eduNEXT/edx-platform that referenced this pull request Sep 14, 2016
* stv/diff/default-store-for-course:
  Remove DEFAULT_STORE_FOR_NEW_COURSE setting
xavierchan pushed a commit to xavierchan/edx-platform-1 that referenced this pull request May 29, 2019
morenol pushed a commit to eduNEXT/edx-platform that referenced this pull request Aug 27, 2019
MoisesGSalas pushed a commit to eduNEXT/edx-platform that referenced this pull request Jan 12, 2022
…le-ar-translations

BB-4977: feat: Compile JS translations for Arabic
Sujeet1379 pushed a commit to chandrudev/edx-platform that referenced this pull request Nov 17, 2022
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.

3 participants