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.

Sign up for GitHub

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

Jump to bottom

Decompose add mentors #773

Merged
merged 6 commits into from
Jun 18, 2024
Merged

Decompose add mentors #773

merged 6 commits into from
Jun 18, 2024

Conversation

Kizr
Copy link
Contributor

@Kizr Kizr commented Jun 17, 2024

Context

The wizard is becoming unmanageable and unmaintainable it, we need to begin refactoring the logic to fix these issues before further development is needed.

Changes proposed in this pull request

  • Add mentors step
  • Use the step rather than the placement for adding mentors
  • Remove redundant logic

Guidance to review

  • Log in as Anne
  • Add a placement
  • Confirm the mentors are saved as expected
  • There should be no discernable changes to the journey

Link to Trello card

Decompose the Placements Build model into Wizard Steps

@Kizr Kizr self-assigned this Jun 17, 2024
@Kizr Kizr requested review from a team as code owners June 17, 2024 15:15
ollietreend
ollietreend reviewed Jun 18, 2024
View reviewed changes
Copy link
Contributor

@ollietreend ollietreend left a comment

Choose a reason for hiding this comment

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

I love it! 😍 This is a really good step in the right direction.

Just a few minor points from me.

spec/wizards/placements/add_placement/steps/mentors_spec.rb Outdated Show resolved Hide resolved
spec/wizards/placements/add_placement/steps/mentors_spec.rb Outdated Show resolved Hide resolved
spec/wizards/placements/add_placement/steps/mentors_spec.rb Outdated Show resolved Hide resolved
spec/wizards/placements/add_placement/steps/mentors_spec.rb Show resolved Hide resolved
config/locales/en/activemodel.yml Outdated Show resolved Hide resolved
ollietreend
ollietreend approved these changes Jun 18, 2024
View reviewed changes
Copy link
Contributor

@ollietreend ollietreend left a comment

Choose a reason for hiding this comment

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

😍

@Kizr Kizr merged commit cd3353f into main Jun 18, 2024
8 checks passed
@Kizr Kizr deleted the ba/decompose_add_mentors branch June 18, 2024 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ollietreend ollietreend ollietreend approved these changes

Assignees

@Kizr Kizr

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Kizr @ollietreend