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

Remove Duplicate Books and improve import-export #1233

Open
mwvolo opened this issue Dec 9, 2021 · 6 comments
Open

Remove Duplicate Books and improve import-export #1233

mwvolo opened this issue Dec 9, 2021 · 6 comments
Assignees
Labels

Comments

@mwvolo
Copy link
Member

mwvolo commented Dec 9, 2021

@oscryan commented on Mon Nov 29 2021

Description

Attempting to duplicate a book page in wagtail in any environment results in a 500 error, even when both the title and the slug are changed. Discussed in https://openstax.slack.com/archives/C69BU01RC/p1638220616221900

Acceptance Criteria

  • Removed Duplicate Book from book menu since it did not work and was a wrapper around import-export
  • Wagtail Copy menu item on the Book menu can be used to copy a book on the same server
  • Import-Export can be used to copy content to other servers. Code was improved to prevent id collisions and notify user of duplicate slug.

To test, you could do these things

  1. Create a new test page on QA or dev and save it. Go to import-export on that server and export. It will download a zip file
  2. Go to import-export on staging and import your zip file. You will have to choose a parent page such as home or subjects depending on where you made your test page
  3. To test Copy, once your test page is in staging, select Copy from the menu and make a copy. You can delete your test pages once your testing is done.
@mwvolo mwvolo added the bug label Dec 9, 2021
@openstaxalina
Copy link

[commenting on this card so I'll see any updates]

@yblock
Copy link
Contributor

yblock commented Jan 27, 2022

There is a new package called Transfer that is the new version of what we're using to import/export pages right now. We should look into moving to this new package to resolve the issue, the current one we use is no longer maintained.

@mwvolo
Copy link
Member Author

mwvolo commented Jan 27, 2022

@openstaxalina @yblock - until we can get to this, we think we can get CMs access to the button for import/export - which should work for book pages, can you comment here on who needs to see this option?

@openstaxalina
Copy link

thank you, @mwharrison ! Colby (@yblock ), Ryan (@oscryan), Jacqueline (@Jacqueline-Yarbrough) and Theresa (@theresamasciale) need this option. We also have a new CM starting on 2/7 who will need it -- we will give you her GitHub username when we have it.

@mwvolo
Copy link
Member Author

mwvolo commented Jan 28, 2022 via email

@mwvolo mwvolo changed the title Cannot Duplicate book page in CMS Migrate wagtail-import-export and duplicate books to use wagtail-transfer May 20, 2022
@edwoodward edwoodward changed the title Migrate wagtail-import-export and duplicate books to use wagtail-transfer Remove Duplicate Books and improve import-export Jun 3, 2022
@mwvolo
Copy link
Member Author

mwvolo commented Jul 26, 2022

@oscryan can you make sure this works on staging? We think we got this working now.
https://staging.openstax.org/admin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants