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.

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

Add cancel function to useResendMessage hook #70

Merged
merged 4 commits into from
Aug 28, 2023
Merged

Conversation

rygine
Copy link
Collaborator

@rygine rygine commented Aug 28, 2023

When a message fails to send, there are 2 options: resend the message, or cancel sending and remove it. This PR adds the cancel functionality to the useResendMessage hook so that developers can cancel the sending of failed messages.

It also renames the resendMessage function to simply resend.

@rygine rygine requested a review from a team August 28, 2023 18:40
@changeset-bot
Copy link

changeset-bot bot commented Aug 28, 2023

🦋 Changeset detected

Latest commit: 9c7055c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@xmtp/react-sdk Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@rygine rygine requested a review from jhaaaa as a code owner August 28, 2023 18:40
nplasterer
nplasterer previously approved these changes Aug 28, 2023
@rygine rygine force-pushed the rygine/update-use-resend branch from ba60b7d to 9c7055c Compare August 28, 2023 18:56
@rygine rygine merged commit e68350b into main Aug 28, 2023
4 checks passed
@rygine rygine deleted the rygine/update-use-resend branch August 28, 2023 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants