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

Move /opportunity pages to /initiatives #2971

Closed
4 tasks done
kristinashu opened this issue Apr 5, 2019 · 23 comments · Fixed by #4382
Closed
4 tasks done

Move /opportunity pages to /initiatives #2971

kristinashu opened this issue Apr 5, 2019 · 23 comments · Fixed by #4382
Assignees
Milestone

Comments

@kristinashu
Copy link

kristinashu commented Apr 5, 2019

At some point we decided to no longer use /opportunity and use /initiatives instead. Now we have projects arbitrarily split between both which is more confusing and complicated for staff and users.

Next steps:

  • confirm that we want to sunset /opportunity
  • move all /opportunity pages to /initiative
  • redirects those pages (one PR)
  • archive /opportunity (follow-up PR after the redirect PR lands)
@kristinashu
Copy link
Author

Looping @anilkanji in on this for approval...

I think this is something Jesse started a while ago. Originally, most staff would create pages under /campaign or /opportunity. But then a decision was made to use /initiatives instead of /opportunity (sorry, I don't have anymore insight into this but assuming it's because Initiatives sounds more accurate and is broader than Opportunity). The intention was to then sunset /opportunity but it just never happened, but only because of neglect and lack of prioritization. So now things are spread across both /opportunity and /initiatives and it's confusing to know which pages should go where. Sooo I think the time has come to move things over and sunset /opportunity.

You ok with that? @alanmoo do you have any more context to add? Would you be able to socialize this with staff once it's happened?

@anilkanji
Copy link
Contributor

To confirm - we're moving over /opportunity pages over to /initiatives?

Sounds good. Alan flagged the need for a broader IA review.

@kristinashu
Copy link
Author

RIP /opportunity

Long live /initiatives

@alanmoo alanmoo modified the milestones: Nov 25, Dec 9 Nov 26, 2019
@alanmoo
Copy link
Contributor

alanmoo commented Nov 26, 2019

I'm going to suggest a manually coded URL redirect rather than doing this in the Wagtail dashboard since there are so many pages. Let's set up the appropriate urls.py file to redirect anything at /opportunity/*/* to /initiative/*/* and then move the pages in the admin shortly after we deploy. (I think a few moments of downtime on those pages is ok)

@alanmoo alanmoo assigned mmmavis and unassigned kristinashu Nov 26, 2019
@alanmoo alanmoo modified the milestones: Dec 9, Icebox Dec 10, 2019
@kristinashu kristinashu modified the milestones: Icebox, Feb 24 Feb 6, 2020
@xmatthewx
Copy link
Contributor

Good plan!

@mmmavis
Copy link
Collaborator

mmmavis commented Feb 12, 2020

@kristinashu is this considered as part of the bigger IA work? If so, it's better that we tackle it along with other IA dev tickets (when they are ready for implementation).

@kristinashu
Copy link
Author

@mmmavis this is something we're wanted to do for a very long time and I think the longer we leave it the more confusing and more work it will be to undo it later. So I don't think it should be blocked by the other IA work. Do you have any concerns about it?

@mmmavis
Copy link
Collaborator

mmmavis commented Feb 24, 2020

@kristinashu can you clarify what you meant by "archive /opportunity"?

@kristinashu
Copy link
Author

I mean that users should no longer be able to create pages under "opportunity" so it shouldn't even show up as an option to a users in the CMS.

@cadecairos cadecairos modified the milestones: Feb 24, Mar 9 Feb 25, 2020
@cadecairos cadecairos removed this from the Mar 9 milestone Mar 10, 2020
@cadecairos cadecairos added this to the Mar 23 milestone Mar 10, 2020
@mmmavis
Copy link
Collaborator

mmmavis commented Mar 11, 2020

PR for redirecting pages under /opportunity to /initiatives has passed code review. We will need to

  1. Go to production CMS and manually move those pages to /initiatives
  2. Then merge and push the PR so old urls get redirected.

@kristinashu do you know when will be a good time to do this?

@kristinashu
Copy link
Author

@mmmavis I can probably move the pages over next week. What to assign the ticket to me until I've done this?

@mmmavis
Copy link
Collaborator

mmmavis commented Mar 12, 2020

@kristinashu done assigning! Right before you start moving the pages next week, can you let the engineering team know so we can be ready to merge and push the redirect PR live?

@kristinashu
Copy link
Author

I've reviewed all the pages and I think they should all be moved directly to /initiatives.

@Pomax @mmmavis are you able to non-manually move all the pages and then redirect the old urls?

@mmmavis
Copy link
Collaborator

mmmavis commented Mar 17, 2020

I can help with the manually move but I'm not comfortable running backend/database commands myself. Is there any concern with moving pages manually?

@kristinashu
Copy link
Author

Just that there are a lot of pages 😭, especially considering the sub- pages and translations. And I thought Pomax said there was a way to do it, but maybe I misunderstood.

@xmatthewx
Copy link
Contributor

@cadecairos - we have 18 wagtail pages and all their subpages to move and set up redirects. We're hoping to not do it manually. Is this something you or Lucie might be able to help do? I wonder if Wagtail has a tool for bulk page management - I've seen that in other cms.

@cadecairos
Copy link

Wagtail has a management command that can be run to move pages: http://docs.wagtail.io/en/v2.0/reference/management_commands.html#move-pages

@Pomax can you coordinate running this command?

@mmmavis
Copy link
Collaborator

mmmavis commented Mar 19, 2020

Yay! 🎉 @Pomax just helped moving all pages under /opportunity to /initiatives! Everything is working fine on prod now.

I'll file a PR for the cleanup work (e.g., update fake data script etc)

@kristinashu
Copy link
Author

w00t! Thank you so much for save me hours and hours of work! Also thank you for helping clean up the CMS and making it easier for all staff to use.

@mmmavis
Copy link
Collaborator

mmmavis commented Mar 23, 2020

Should we remove the opportunities mini site name space from production CMS?

image

@kristinashu
Copy link
Author

Yes! Please remove it.

@mmmavis
Copy link
Collaborator

mmmavis commented Mar 24, 2020

Done! Thanks all! 🎉

@mmmavis mmmavis closed this as completed Mar 24, 2020
@xmatthewx
Copy link
Contributor

awesome! 🥇 thanks all for making that happen!

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

Successfully merging a pull request may close this issue.

6 participants