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

Rename accredited partnership concepts #4808

Closed
wants to merge 8 commits into from

Conversation

inulty-dfe
Copy link
Contributor

@inulty-dfe inulty-dfe commented Jan 9, 2025

Context

There are two main areas this work focusses on

  1. Training -> Accredited partnerships
  2. Course ratification

Partnerships

A training provider must have established a partnership with an Accredited provider before they can assign that accredited partner to be the ratifying partner for their training course.

a) link in the navigation bar "Accredited partnerships" brings the user to the index of the training providers existing partnerships.
b) There is a setup flow for adding a new accredited partnership. Within this flow the languages has changed from "accredited provider" to "accredited partner".

Ratifying partner

a) Anywhere the UI displays reference to the accredited provider that is ratifying the course, the language has changed from "accrediting provider" to "ratifying partner".

Help pages

http://publish.localhost:3001/how-to-use-this-service
https://qa.publish-teacher-training-courses.service.gov.uk/how-to-use-this-service/change-an-accredited-provider-relationship

For all these changes there are subtle differences between the options available.

For example, should the accredited provider of a course be the "ratifying provider" or the "ratifying partner"

Label Meaning
Providers All providers
Training provider A provider that runs ITT training courses
Accredited provider A provider that is accredited by DfE
Training partner The training provider in a provider partnership association
Accredited partner The accredited provider in a provider partnership association
Ratifying partner The Accredited partner that ratifies the course of a Training partner

The wording in specs still needs to be changed.
For example

  scenario 'i can edit accredited providers on the index page' do

Needs to change to accredited partners. This is WIP until all the wording is updated.

Changes proposed in this pull request

Update all the UI labels and text to accurately convey the concepts above.

Guidance to review

Accredited partners V Accredited partnerships

Trello

https://trello.com/c/kdNdMBgS/278-tech-debt-rename-concepts-in-the-application

@inulty-dfe inulty-dfe added the deploy A Review App will be created for PRs with this label label Jan 9, 2025
@inulty-dfe inulty-dfe force-pushed the im/338-tech-debt-rename-partnership-concepts branch from 762e7f0 to f3179a2 Compare January 9, 2025 15:18
@inulty-dfe inulty-dfe marked this pull request as ready for review January 10, 2025 08:56
@inulty-dfe inulty-dfe requested a review from a team as a code owner January 10, 2025 08:56
@inulty-dfe inulty-dfe added WIP Work in progress, not yet ready for review wontfix This will not be worked on labels Jan 13, 2025
@inulty-dfe
Copy link
Contributor Author

The team has decided not to change the wording around accredited providers at this point

@inulty-dfe inulty-dfe closed this Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy A Review App will be created for PRs with this label WIP Work in progress, not yet ready for review wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant