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

fix: simplified mock oauth with mswjs #183

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Conversation

eddiejaoude
Copy link
Member

Fixes Issue

We are using mswjs for mocking github API requests, while writing a blog on it, I thought it might simplify our code if we used the same process for mocking authenticated users during testing (and for offline development or when there is no github oauth app)

Changes proposed

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants