Skip to content

Profiles: Professional Memberships (removed)#8675

Open
starswan wants to merge 4 commits intomainfrom
remove-profile-professional-memberships
Open

Profiles: Professional Memberships (removed)#8675
starswan wants to merge 4 commits intomainfrom
remove-profile-professional-memberships

Conversation

@starswan
Copy link
Copy Markdown
Contributor

Trello card URL

https://trello.com/c/WXACw5a1/2674-profile-professional-body-memberships-journey

Changes in this PR:

Remove professional memberships from profiles

Screenshots of UI changes:

Before

After

@starswan starswan marked this pull request as draft March 30, 2026 10:54
@starswan starswan marked this pull request as ready for review April 1, 2026 12:28
@starswan starswan added the deploy label Apr 7, 2026
@starswan starswan force-pushed the remove-profile-professional-memberships branch from c10495d to 8a4c6b7 Compare April 7, 2026 14:29
@starswan starswan force-pushed the remove-profile-professional-memberships branch from 8a4c6b7 to 623b0b4 Compare April 7, 2026 15:08
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

Review app deployed to https://teaching-vacancies-review-pr-8675.test.teacherservices.cloud on AKS

@starswan starswan force-pushed the remove-profile-professional-memberships branch from 623b0b4 to 3b7786b Compare April 13, 2026 14:41
ProfessionalBodyMembership.transaction do
pbm_batch.each(&:destroy!)
end
end
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Proposal: Couldn't we leave the schema migration for a second PR?

  1. This PR:
  • Interface & Flow changes.
  • Jobseekers can no longer add or see listed Professional Body Memberships.
  • Rake task with above code, removing any existing Professional Body Membership belonging to Jobseeker Profiles.
  1. Second PR:
  • Schema migration
  • Removal of model association
  • Analytics, etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants