Skip to content

Update routes to Strapi #2328

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

Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

Update routes to Strapi #2328

wants to merge 20 commits into from

Conversation

msquance-stem
Copy link
Contributor

Status

Review progress:

  • Browser tested
  • Front-end review completed
  • Tech review completed

What's changed?

  • Remove pedagogy page

Steps to perform after deploying to production

If the production environment requires any extra work after this PR has been deployed detail it here. This could be running a Rake task, migrating a DB table, or upgrading a Gem. That kind of thing.

@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 March 4, 2025 14:31 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 March 4, 2025 15:08 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 March 4, 2025 15:47 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 March 21, 2025 10:45 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 March 21, 2025 11:49 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 March 21, 2025 11:53 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 March 21, 2025 12:15 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 March 21, 2025 12:24 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 March 21, 2025 12:34 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 March 21, 2025 12:57 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 March 21, 2025 13:03 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 March 21, 2025 13:34 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 March 31, 2025 08:50 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 April 2, 2025 12:47 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 April 3, 2025 11:38 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 April 7, 2025 14:33 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 April 17, 2025 09:07 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 April 17, 2025 10:55 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 April 17, 2025 15:19 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 April 17, 2025 16:04 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2328 April 17, 2025 16:13 Inactive
@msquance-stem
Copy link
Contributor Author

This should be almost good to go once @A-Wheeto has completed the enrolment resource card component.

One thing to double check before go live

  • About has been swapped over to Strapi, but it doesn't appear to have had comms sign off according to the spreadsheet. So this should be checked

We have NOT migrated the careers support page. This is due to having to make a benchmark variants of all the components, and this seems excessive for one page.

@A-Wheeto A-Wheeto force-pushed the 2993-page-previews branch from a12c045 to a113d7d Compare May 12, 2025 09:05
@A-Wheeto A-Wheeto changed the title allow preview of pedagogy page Update routes to Strapi May 16, 2025
@A-Wheeto A-Wheeto force-pushed the 2993-page-previews branch from 0f2a76f to 5639b67 Compare May 16, 2025 08:32
@A-Wheeto A-Wheeto temporarily deployed to teachcomputing-pr-2328 May 16, 2025 08:32 Inactive
Copy link

@markjs markjs left a comment

Choose a reason for hiding this comment

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

Looks good, always love to see a PR that's mostly deletions. I would say the routes that are no longer needed in the routes file would be much better deleted than commented out if there is no expectation to need them again any time soon 👍

Switch secondary-teachers to CMS
Updating site searchable tests
@tc-deploybot tc-deploybot temporarily deployed to teachcomputing-pr-2328 May 27, 2025 10:18 Inactive
Removing padding from secondary question bank component for better placement on the page
Fixing check for aside icons in aside component as this was returning true falsely
@A-Wheeto A-Wheeto temporarily deployed to teachcomputing-pr-2328 May 28, 2025 12:01 Inactive
…nent that was matching a class with icon, but wasn't testing for the correct icon class
@A-Wheeto A-Wheeto temporarily deployed to teachcomputing-pr-2328 May 28, 2025 12:58 Inactive
Copy link

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.

4 participants