Skip to content

feat: edit experience issue 8 clean#39

Closed
aboderinsamuel wants to merge 4 commits into
MLH-Fellowship:mainfrom
aboderinsamuel:edit-experience-issue-8-clean
Closed

feat: edit experience issue 8 clean#39
aboderinsamuel wants to merge 4 commits into
MLH-Fellowship:mainfrom
aboderinsamuel:edit-experience-issue-8-clean

Conversation

@aboderinsamuel

Copy link
Copy Markdown
Member

Summary

This PR adds support for updating an existing Experience entry using its index (ID) via a PUT request to /resume/experience.

Related Issue

What’s Included

  • PUT handling on /resume/experience that:
    • Validates the id
    • Validates required fields
    • Updates the experience at the given index
  • Refactored response flow to avoid too many return statements (pylint compliance)
  • New pytest that:
    • Creates an experience
    • Updates it via PUT
    • Verifies the list reflects the updated values

API Behavior

Request:

@JocelynVelarde JocelynVelarde left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

please resolve the merge conflicts

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