Skip to content

Fixing back reference to recurring event when rescheduled#3967

Merged
emrysal merged 6 commits intomainfrom
fix/recurring-event-rescheduled
Aug 25, 2022
Merged

Fixing back reference to recurring event when rescheduled#3967
emrysal merged 6 commits intomainfrom
fix/recurring-event-rescheduled

Conversation

@leog
Copy link
Copy Markdown
Contributor

@leog leog commented Aug 25, 2022

What does this PR do?

When an event part of a recurring booking is rescheduled, the reference to the original recurring booking was lost, making the booking not part of the series as originally set.

Environment: Staging(main branch) / Production

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

Book a recurring event, have one instance rescheduled, see the recurring events count in the Recurring booking tab stay the same.

@leog leog added the 🐛 bug Something isn't working label Aug 25, 2022
@leog leog added this to the v.2.0 milestone Aug 25, 2022
@leog leog requested a review from a team August 25, 2022 14:03
@leog leog self-assigned this Aug 25, 2022
@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 25, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Aug 25, 2022 at 2:29PM (UTC)
1 Ignored Deployment
Name Status Preview Updated
swagger ⬜️ Ignored (Inspect) Aug 25, 2022 at 2:29PM (UTC)

@emrysal emrysal enabled auto-merge (squash) August 25, 2022 14:15
@PeerRich PeerRich added automerge ♻️ autoupdate tells kodiak to keep this branch up-to-date labels Aug 25, 2022
@emrysal emrysal merged commit 9c3cea5 into main Aug 25, 2022
@emrysal emrysal deleted the fix/recurring-event-rescheduled branch August 25, 2022 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge ♻️ autoupdate tells kodiak to keep this branch up-to-date 🐛 bug Something isn't working

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants