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

Bugfix/pdct 313 #36

Merged
merged 17 commits into from
Oct 24, 2023
Merged

Bugfix/pdct 313 #36

merged 17 commits into from
Oct 24, 2023

Conversation

katybaulch
Copy link
Contributor

Description

Please include:

  • a summary of the changes
  • links to any related issue/ticket
  • any additional relevant motivation and context
  • details of any dependency updates that are required for this change

Type of change

Please select the option(s) below that are most relevant:

  • Bug fix
  • New feature
  • Breaking change

How Has This Been Tested?

Please describe the tests that you added to verify your changes.

Reviewer Checklist

  • The PR represents a single feature (small driveby fixes are also ok)
  • The PR includes tests that are sufficient for the level of risk
  • The code is sufficiently commented, particularly in hard-to-understand areas
  • Any required documentation updates have been made
  • Any TODOs added are captured in future tickets
  • No FIXMEs remain

@katybaulch
PDCT-311 Added endpoint and tests for updating family events.
22eb526
@katybaulch
PDCT-311 Remove dev debug logs.
fbb1c5f
@katybaulch
PDCT-313 Added deleted to family EventStatus model.
059766d
@katybaulch
PDCT-313 Added delete endpoint for family events.
6e6837f
@katybaulch
PDCT-313 Removed TODO.
2f4339c
@katybaulch
PDCT-313 Added unit and integration tests for deleting family events.
6dd2f52
@katybaulch
Merge branch 'main' into PDCT-313-endpoint-for-deleting-events
83c7295
@katybaulch
PDCT-313 Add deleted to event status enum for test db setup.
6f92864
@katybaulch
Merge branch 'main' into PDCT-313-endpoint-for-deleting-events
6b3eca7
@katybaulch
PDCT-313 Implement DB delete.
3b7d694
@katybaulch
Driveby remove admin permissions for MVP.
a35ddfb
@katybaulch
Driveby skip 403 when user not admin tests for MVP.
ebae544
@katybaulch
PDCT-313 FRemoved duplicate assertions.
a7a8055
@katybaulch
Merge branch 'main' into PDCT-313-endpoint-for-deleting-events
38532f9
@katybaulch
PDCT-313 Removed hanging Event deleted status
a8c64cc
@katybaulch
PDCT-313 Updated dto to dict.
7d1a079
@katybaulch
Merge branch 'main' into bugfix/PDCT-313
616c5c2
@linear
Copy link

linear bot commented Oct 24, 2023

PDCT-313 Endpoint for deleting events

@katybaulch katybaulch requested a review from diversemix October 24, 2023 17:21
@diversemix diversemix merged commit 33b7a25 into main Oct 24, 2023
2 checks passed
@diversemix diversemix deleted the bugfix/PDCT-313 branch October 24, 2023 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@diversemix diversemix diversemix 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
@katybaulch @diversemix