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

Add teacher degree apprenticeship filter to Manage #9406

Merged
merged 5 commits into from
Jun 5, 2024

Conversation

tomas-stefano
Copy link
Collaborator

Context

We need to make changes to Manage to allow providers to view applications with the a new type “Teacher degree apprenticeship (TDA) with QTS”.

Changes proposed in this pull request

Screenshot 2024-05-28 at 16 17 36
  • Added a new filter for TDA courses (when apply the filters when user select)
Course type

[ ] Postgraduate courses
[ ] Teaching degree apprenticeship (TDA) courses

Guidance to review

If you want to test:

  1. Enable the feature flag in the review app
  2. Create a TDA course (see spec/factories/course trait TDA)
  3. Apply for a TDA course
  4. Then filter for TDA and non TDA applications in Manage

Trello card

https://trello.com/c/HIsIqv1q/1728-add-filters-to-manage-for-tda

@tomas-stefano tomas-stefano added the deploy_v2 Deploy the review app to AKS label May 28, 2024
@tomas-stefano tomas-stefano requested a review from a team May 28, 2024 15:19
Copy link
Contributor

Database-level enum changes detected

Please include a data migration for these attributes and values:

12c12
< <enum> Course.program_type: ["HE", "HES", "SD", "SS", "SC", "SSC", "TA"]
---
> <enum> Course.program_type: ["HE", "HES", "SD", "SS", "SC", "SSC", "TA", "TDA"]

@github-actions github-actions bot temporarily deployed to review_aks-9406 May 28, 2024 15:24 Destroyed
@github-actions github-actions bot temporarily deployed to review_aks-9406 May 28, 2024 15:34 Destroyed
trait :teacher_degree_apprenticeship do
apprenticeship
full_time
description { 'Teacher degree apprenticeship with QTS full time teaching apprenticeship' }
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
description { 'Teacher degree apprenticeship with QTS full time teaching apprenticeship' }
description { 'Teaching degree apprenticeship with QTS full time teaching apprenticeship' }

Is it Teaching degree apprenticeship or Teacher degree apprenticeship? We are not consistent. it may be too late to change in some places (eg the programme_type), but we should make an effort where possible.

Copy link
Collaborator Author

@tomas-stefano tomas-stefano May 29, 2024

Choose a reason for hiding this comment

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

Actually this comes from the content designer and from the prototype. I might keep as it is

@github-actions github-actions bot temporarily deployed to review_aks-9406 May 29, 2024 14:44 Destroyed
@tomas-stefano tomas-stefano marked this pull request as ready for review May 29, 2024 14:52
@tomas-stefano
Copy link
Collaborator Author

Copy link
Contributor

@dcyoung-dev dcyoung-dev left a comment

Choose a reason for hiding this comment

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

A couple of suggestions wordings 👌

@github-actions github-actions bot temporarily deployed to review_aks-9406 June 4, 2024 10:35 Destroyed
Copy link
Collaborator

@avinhurry avinhurry left a comment

Choose a reason for hiding this comment

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

A couple of minor questions/suggestions but looks good, nice work 👍🏼

end

it 'does include the course type filter' do
expect(filter.filters.size).to be(7)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this adding value? If we add another filter this will fail?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

i just maintaned the same test that was there and added the tda filter. I think I will maintain this.

@github-actions github-actions bot temporarily deployed to review_aks-9406 June 5, 2024 09:18 Destroyed
@tomas-stefano tomas-stefano merged commit 61b4266 into main Jun 5, 2024
24 checks passed
@tomas-stefano tomas-stefano deleted the 1728-add-filters-to-manage-for-tda branch June 5, 2024 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy_v2 Deploy the review app to AKS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants