Skip to content

Feature/Delete an existing experience #38

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
Merged
merged 5 commits into from
Feb 9, 2025
Merged

Feature/Delete an existing experience #38

merged 5 commits into from
Feb 9, 2025

Conversation

FortunateOmonuwa
Copy link
Member

  • Implemented a new DELETE endpoint for deleting existing experiences that returns a JSON response:
    • A 200 status code with a success message when an experience is successfully deleted.
    • 404 Not Found status code If the provided ID doesn't match any resource.
    • 500 Internal Server Error: If an exception occurs.

@Pradyuman7

@FortunateOmonuwa FortunateOmonuwa changed the title add implementation for deleting existing experience feature/delete an existing experience Feb 6, 2025
@FortunateOmonuwa FortunateOmonuwa changed the title feature/delete an existing experience Delete an existing experience Feb 6, 2025
@FortunateOmonuwa FortunateOmonuwa changed the title Delete an existing experience Feature/Delete an existing experience Feb 6, 2025
Pradyuman7
Pradyuman7 reviewed Feb 6, 2025
View reviewed changes
Pradyuman7
Pradyuman7 reviewed Feb 6, 2025
View reviewed changes
Pradyuman7
Pradyuman7 reviewed Feb 7, 2025
View reviewed changes
Copy link
Collaborator

@Pradyuman7 Pradyuman7 left a comment

Choose a reason for hiding this comment

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

Just one quick comment otherwise LGTM 💯

Pradyuman7
Pradyuman7 approved these changes Feb 9, 2025
View reviewed changes
Copy link
Collaborator

@Pradyuman7 Pradyuman7 left a comment

Choose a reason for hiding this comment

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

LGTM 🔥 please resolve the conflicts and then feel free to merge

@Pradyuman7 Pradyuman7 merged commit 3d10db0 into MLH-Fellowship:main Feb 9, 2025
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Pradyuman7 Pradyuman7 Pradyuman7 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@FortunateOmonuwa @Pradyuman7