Skip to content

Fixing issues around workflows#6991

Merged
PeerRich merged 4 commits intomainfrom
fix/reminders-for-cancelled-events
Feb 13, 2023
Merged

Fixing issues around workflows#6991
PeerRich merged 4 commits intomainfrom
fix/reminders-for-cancelled-events

Conversation

@CarinaWolli
Copy link
Copy Markdown
Member

What does this PR do?

Fixes issue that workflow email reminders of canceled bookings were still sent.
Fixes issue that bookings that requested rescheduling still sent SMS and email reminders.
Also, fixes the issue that when editing a workflow the data gets lost when clicking outside of the workflow.

Fixes #6973
Fixes #6905
Fixes #6956

Environment: Staging(main branch) / Production

Type of change

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

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 9, 2023

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

Name Status Preview Comments Updated
cal ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 10, 2023 at 10:03AM (UTC)
ui ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 10, 2023 at 10:03AM (UTC)

},
});

await client.request({
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

SendGrid saves canceled emails (until the scheduled date) and our max limit of saved entries was reached. So from now on we delete them right after canceling. I deleted all scheduled_sends from SendGrid, so it should already work again

},
},
steps: true,
steps: {
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

this fixes that added steps were in wrong order on first load

});

// delete scheduled jobs of cancelled bookings
cancelScheduledJobs(bookingToReschedule);
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

cancel scheduledJobs and all workflow reminders for the booking if rescheudling is requested

@CarinaWolli CarinaWolli requested a review from a team February 9, 2023 22:01
Copy link
Copy Markdown
Contributor

@emrysal emrysal left a comment

Choose a reason for hiding this comment

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

Nice, simple fixes 👍

@PeerRich PeerRich enabled auto-merge (squash) February 10, 2023 09:54
@PeerRich PeerRich merged commit 1cae2b6 into main Feb 13, 2023
@PeerRich PeerRich deleted the fix/reminders-for-cancelled-events branch February 13, 2023 12:39
@PeerRich PeerRich added the core area: core, team members only label Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only

Projects

None yet

3 participants