Skip to content

Conversation

@ichuang
Copy link
Contributor

@ichuang ichuang commented Aug 16, 2013

It is useful for sysadmin staff to be able to:

  • Purge the mongodb of course fragments, which match course id's but are not deleted through the normal django management command (which deletes only entries linked in the course tree). We've seen such course content corruption in the database happen several times, associated with course teams importing .tar.gz files with errors.
  • Load course from a git url via a web interface
  • Get JSON dump of course content directly from the mongodb (eg for quick and exact transfer of course modules to another Studio instance).

This PR provides those features, via a rudimentary but functional dashboard.

No tests provided.

@jtauber
Copy link
Contributor

jtauber commented Aug 21, 2013

what is the relationship between this and #654 ?

@ichuang
Copy link
Contributor Author

ichuang commented Aug 21, 2013

This one is for the CMS; #654 is for the LMS

Copy link

Choose a reason for hiding this comment

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

ensure_csrf?

@singingwolfboy
Copy link
Contributor

What's the status of this pull request? It looks like it's gone stale.

@ichuang
Copy link
Contributor Author

ichuang commented Oct 8, 2013

This feature is in production at MIT, and has been for about two months.

@chrisndodge
Copy link
Contributor

Rebased to resolve conflicts

@chrisndodge
Copy link
Contributor

@ichuang - added I18N support. Did a quick spot check to see if I broke anything. Can you verify? Some one the UI looks pretty wonky.... can you verify that's the way it should look.

Also, I can't run the delete function - I get this exception:

[Errno 2] No such file or directory: u'DATA-BACKUP/course-ppp-dump-Wed_Oct_16_21:23:45_2013.json'

Seems like we need to make that path configurable

@ichuang
Copy link
Contributor Author

ichuang commented Oct 17, 2013

added settings config for DELETED_COURSE_BACKUPS_DIR

Does this run for you now?

@chrisndodge
Copy link
Contributor

settings.getattr(...) should be getattr(settings,...). fixed.

delete seems to work now.

@ichuang
Copy link
Contributor Author

ichuang commented Oct 17, 2013

thanks for the fix

@chrisndodge
Copy link
Contributor

Can you verify the UI is what you want? When I delete course, the 'yes' confirmation is way to the right of the page

@ichuang
Copy link
Contributor Author

ichuang commented Oct 17, 2013

that's deliberate - the "yes" is supposed to be hard to accidentally click.

Just took a look. The extra-large edX studio logo is a bit wonky, probably css thing, but acceptable. All the UI elements are functional, albeit plain. It's what we made.

@chrisndodge
Copy link
Contributor

OK, I have a bunch of meeting in the morning so I should be able to finish this up later up this afternoon..

@cpennington
Copy link
Contributor

This needs tests, so that we can be sure that it stays working as other functionality in the platform changes.

Copy link
Contributor

Choose a reason for hiding this comment

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

This is an odd docstring: a) It isn't attached to anything (module, class, function). b) It doesn't actually include any description. If it's intended as a comment, please just use #.

@cpennington
Copy link
Contributor

This has a number of obvious pylint and pep8 violations. Please run rake quality to see the list of newly added violations.

@chrisndodge
Copy link
Contributor

needed to rebase again.

@chrisndodge
Copy link
Contributor

Added some stuff to address @cpennington's concerns. I also put this page on a MIT_FEATURES configuration switch as edX wouldn't want this in production mainly because of the potential for accidental data loss - we have a Delete Course story which have a lot of safeguards.

Will do a few more items today.

Copy link
Contributor

Choose a reason for hiding this comment

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

Since this view is @login_required, request.user will always be set.

@carsongee
Copy link
Contributor

With the acceptance of the LMS sysadmin dashboard, this is no longer needed. The only thing from it we need to take a look at is that it's more broad delete function removes orphaned mongo nodes and doesn't get caught up by duplicate mongo store items. I've added it to my task list to look into that.

@carsongee carsongee closed this Jan 17, 2014
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
show full sidebar title; blockquote and list styles for comments
@benpatterson benpatterson deleted the feature/ichuang/studio-sysadmin-dashboard branch January 21, 2015 13:13
jfavellar90 added a commit to eduNEXT/edx-platform that referenced this pull request Apr 20, 2018
…0.2.3

Bumping version of rocket chat xblock to 0.2.3
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.

8 participants