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

Delete school experience helped me switch blog #3913

Closed
wants to merge 21 commits into from

Conversation

MaxineCl
Copy link
Contributor

@MaxineCl MaxineCl commented Mar 25, 2024

Trello card

https://trello.com/c/L7AtyEFx/2181-delete-and-redirect-blog-post-on-school-experience-helped-me-switch

Context

We want to delete this blog post https://getintoteaching.education.gov.uk/blog/school-experience-helped-me-decide-to-switch and add a quote from it to the information page on Get School Experience experience https://getintoteaching.education.gov.uk/is-teaching-right-for-me/get-school-experience

Changes proposed in this pull request

  • add quote into Get school experience page
  • delete school experience helped me switch blog post
  • delete blog post from any related links
  • set up redirect for blog post to take users to Get school experience page

Guidance to review

Copy link

Copy link

@@ -45,6 +49,10 @@ During your experience, you’ll get to do things like:
- speak to teachers and meet pupils
- learn more about teacher training - including the application and interview process

<br />
Copy link
Contributor

Choose a reason for hiding this comment

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

You shouldn't need this break before the quote? It's creating a lot of white space

@@ -45,6 +49,10 @@ During your experience, you’ll get to do things like:
- speak to teachers and meet pupils
- learn more about teacher training - including the application and interview process

<br />
$q-Zainab$
<br />
Copy link
Contributor

Choose a reason for hiding this comment

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

You shouldn't need this break after the quote? It's creating a lot of white space

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No problem. I'll delete the break. I copied it from the quote on another page so I don't know if that may need the spacing removing from it too?

@MaxineCl MaxineCl closed this Mar 26, 2024
@MaxineCl MaxineCl deleted the delete-school-experience-helped-me-switch-blog branch March 26, 2024 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants