openstax-cms icon indicating copy to clipboard operation
openstax-cms copied to clipboard

Remove Duplicate Books and improve import-export

Open mwvolo opened this issue 4 years ago • 6 comments

@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 avatar Dec 09 '21 16:12 mwvolo

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

openstaxalina avatar Jan 24 '22 20:01 openstaxalina

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.

yblock avatar Jan 27 '22 16:01 yblock

@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?

mwvolo avatar Jan 27 '22 19:01 mwvolo

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.

openstaxalina avatar Jan 27 '22 20:01 openstaxalina

All these users should be good to go! Try to use the import/export instead of the book duplicate feature until we can take a deeper look into that.

On Jan 27, 2022, at 2:13 PM, Alina Marie Slavik @.***> wrote:

thank you, @mwharrison https://github.com/mwharrison ! Colby @.*** https://github.com/yblock ), Ryan @.*** https://github.com/oscryan), Jacqueline @.*** https://github.com/Jacqueline-Yarbrough) and Theresa @.*** https://github.com/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.

— Reply to this email directly, view it on GitHub https://github.com/openstax/openstax-cms/issues/1233#issuecomment-1023599875, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA5ZP3FOLWCGBIMWILK3HDTUYGRM7ANCNFSM5JW4HLVQ. You are receiving this because you were mentioned.

mwvolo avatar Jan 28 '22 15:01 mwvolo

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

mwvolo avatar Jul 26 '22 15:07 mwvolo