-
Notifications
You must be signed in to change notification settings - Fork 78
Add documentation on library backup & restore #1335
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
Conversation
|
Thanks @sarina, this looks great to me. I reviewed the documentation and they instructions are clear and cover the details well. The one area of feedback I have is around naming-- because Content Libraries support a separate "Import" feature that is a course → library content import, I would recommend avoiding naming overlap for this documentation page and its sections. As of the last time we discussed this, I think we wanted to use "Backup to local archive" and "Create from archive" or "Restore from archive" to avoid the naming overlap. @jmakowski1123 may have additional thoughts (also CC @edschema) The naming overlap between different platform features (import / export / backup / restore / migrate) is confusing, and I think it would be worth standardizing at some point. For now, I think limiting the overlap will help reduce possible confusion (especially since the Library menu Tools → Import is for course to library content import, not restore-from-backup). |
sdaitzman
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great to me, requesting some very minor changes to phrasing to avoid overlap between create-from-archive and course import tools
source/educators/how-tos/course_development/import_export_library.rst
Outdated
Show resolved
Hide resolved
source/educators/how-tos/course_development/import_export_library.rst
Outdated
Show resolved
Hide resolved
|
|
||
| #. When the import process finishes, you'll be taken to your new library. | ||
|
|
||
| .. seealso:: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder about also linking to migration and course import documentation here, since those are related concepts that enable users to move content back and forth between different areas of the platform
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you have links to the documentation you're mentioning?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking into whether we have documentation for those areas; unfortunately I think they may be additional gaps
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
By "migration" do you mean these? #1340
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise, could you possibly make some issues for the missing docs and approve this PR, and we can write up the missing docs & link them here as they're written
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added this reference, it seems like #1348 isn't ready for documentation yet so we don't need to reference it yet, and we can move forward with merging this PR. Would you agree?
Agree we don't want duplicative naming for Import. We did land on "Backup to local archive" and "Restore from archive". |
ba943c4 to
f6b39a4
Compare
|
@jmakowski1123 I had already adjusted the language to use Backup/Restore, please let me know if you think I missed something! |
https://docsopenedxorg--1335.org.readthedocs.build/en/1335/educators/how-tos/course_development/import_export_library.html