Skip to content
This repository has been archived by the owner on Mar 10, 2024. It is now read-only.

feat: always upsert sequence state #2024

Merged
merged 2 commits into from
Dec 5, 2023
Merged

feat: always upsert sequence state #2024

merged 2 commits into from
Dec 5, 2023

Conversation

asdfryan
Copy link
Contributor

@asdfryan asdfryan commented Dec 4, 2023

Tested locally

@vercel Vercel
Copy link

vercel bot commented Dec 4, 2023 •
edited

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

Name Status Preview Comments Updated (UTC)
management-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 5, 2023 0:35am
supaglue-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 5, 2023 0:35am

tomkit
tomkit approved these changes Dec 5, 2023
View reviewed changes
Copy link
Contributor

@tomkit tomkit left a comment

Choose a reason for hiding this comment

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

nit: some grammarly fixes

openapi/v2/engagement/paths/sequence_states.yaml Outdated Show resolved Hide resolved
@asdfryan asdfryan merged commit f786ca6 into main Dec 5, 2023
11 checks passed
@asdfryan asdfryan deleted the upsert_sequence_state branch December 5, 2023 00:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tomkit tomkit tomkit approved these changes

@lucasmarshall lucasmarshall Awaiting requested review from lucasmarshall

@tonyxiao tonyxiao Awaiting requested review from tonyxiao

@george-xing george-xing Awaiting requested review from george-xing george-xing is a code owner

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

Successfully merging this pull request may close these issues.

None yet
2 participants
@asdfryan @tomkit