Skip to content

Add new services#46

Closed
duncanjbrown wants to merge 2 commits intomainfrom
new-services
Closed

Add new services#46
duncanjbrown wants to merge 2 commits intomainfrom
new-services

Conversation

@duncanjbrown
Copy link
Copy Markdown
Contributor

@duncanjbrown duncanjbrown commented Jul 3, 2023

  • Teacher pay calculator
  • Refer a teacher for serious misconduct
  • Check the Children's barred list
  • Access your teaching qualifications
  • Get a teacher relocation payment

Also stop docs breaking builds when they have non-breaking spaces in the title

- Teacher pay calculator
- Refer a teacher for serious misconduct
- Check the Children's barred list
- Access your teaching qualifications
- Get a teacher relocation payment
- consistently convert markdown contents to UTF-8 so we don't end up
  with ASCII in unexpected places
- handle (i.e. remove) non-breaking spaces in the UTF-8 string
Copy link
Copy Markdown

@gpeng gpeng left a comment

Choose a reason for hiding this comment

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

Thanks for doing this @duncanjbrown. Just a couple of comments. They seem to be in the list already but aren't showing on the deployed site 🤷 For 'Access your teaching qualifications' (and I suppose the same applies for Apply for ITT and Publish) I wonder is there any value in us being able to list the other service names that are in those repos. (I don't mean in this PR, just thinking aloud).

repo_name: "DFE-Digital/teacher-pay-calculator"
language: 'ruby'
docsets: []
- name: "Refer a teacher for serious misconduct"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This one is already on 120 with a slightly different name. I think the existing entry is correct.

language: 'ruby'
docsets:
- path: 'docs'
- name: "Access your teaching qualifications"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This one is on 114.

@duncanjbrown
Copy link
Copy Markdown
Contributor Author

Superseded by #48

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.

2 participants