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

Cleanup on user fetch failure #337

Merged
merged 4 commits into from
Jun 27, 2023
Merged

Conversation

teimurjan
Copy link
Contributor

What's done

  • Added cleanup on user fetch failure.
  • Allowed to override API Client headers.
    • We need when a custom getUser function is provided.

How to test

Tasks

  • Have you written tests for new code (if applicable)?
  • Have you tested the changes on all the platforms?
    • iOS
    • Android
    • Web
  • Have you added stories to demonstrate the new functionality (if there is any)?

Demo (screenshots, recordings, etc.)

@teimurjan teimurjan requested a review from diegonzs-roll June 27, 2023 10:39
@teimurjan teimurjan self-assigned this Jun 27, 2023
@changeset-bot
Copy link

changeset-bot bot commented Jun 27, 2023

🦋 Changeset detected

Latest commit: 0871c5b

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

This PR includes changesets to release 3 packages
Name Type
@roll-network/api-client Minor
@roll-network/auth-sdk Minor
@roll-network/session-manager Major

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

@teimurjan teimurjan merged commit 83150bd into main Jun 27, 2023
@teimurjan teimurjan deleted the cleanup-on-user-fetch-failure branch June 27, 2023 10:47
@github-actions github-actions bot mentioned this pull request Jun 27, 2023
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