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.

Sign up for GitHub

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

Jump to bottom

Auth0 Integration #378

Merged
merged 68 commits into from Feb 6, 2023
Merged

Auth0 Integration #378

merged 68 commits into from Feb 6, 2023

Conversation

oo-bldrs
Copy link
Contributor

@oo-bldrs oo-bldrs commented Sep 14, 2022 •
edited

PR to integrate with Auth0 for Github OIDC.

This also replaces isRunningLocally with a check for whether we're operating in a testing environment (via the NODE_ENV environment variable value).

@netlify
Copy link

netlify bot commented Sep 14, 2022 •
edited

✅ Deploy Preview for bldrs-share ready!

Name Link
🔨 Latest commit d1fc88d
🔍 Latest deploy log https://app.netlify.com/sites/bldrs-share/deploys/63e1295096c4d90008d8f98d
😎 Deploy Preview https://deploy-preview-378--bldrs-share.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@oo-bldrs
Update yarn lockfile
b937a41
@oo-bldrs
Default arguments to empty object
6a60e92
@oo-bldrs
Add Accept: header for Github API calls
141fdd3
@oo-bldrs
Add HTTP request header for authorization
7260ca2
@oo-bldrs
Use access token to make request for comments
99235d5
@oo-bldrs
Do not raise a popup for retrieving Auth0 access token
1555aaa
@oo-bldrs
Send access token on remote call
24b1e09 
TODO: Refactor into single function to avoid eliminate duplication
@oo-bldrs
Fix arguments passed to Octokit
5e10a5b
@oo-bldrs
Allow GitHub base URL to be set via an environment variable
df76834
@oo-bldrs
No-op commit
41e5166
@oo-bldrs
Merge branch 'main' into auth0-integration
6bdb458 
# Conflicts:
#	package.json
#	src/BaseRoutes.jsx
#	src/Components/IssuesControl.jsx
#	src/Components/OperationsGroup.jsx
#	yarn.lock
@oo-bldrs
Remove commented out code (Octokit instantiation)
6a371dd
@oo-bldrs
Update esbuild configuration to pass along null if no redirect URI pr…
8745166 
…ovided
@oo-bldrs
Update size of GitHub avatar to be in line with new UI
567de0d
@oo-bldrs
Update dependency lock file
78b0a3c
@oo-bldrs
Merge branch 'main' into auth0-integration
48e8383 
# Conflicts:
#	src/Components/issues/Issues.jsx
#	src/utils/GitHub.js
@oo-bldrs
Correct capitalization of GitHub
2d4c530
@oo-bldrs
Create store member for GitHub access token
324a031
@oo-bldrs oo-bldrs marked this pull request as draft January 18, 2023 20:54
@oo-bldrs oo-bldrs marked this pull request as ready for review January 24, 2023 16:26
@pablo-mayrgundter
Copy link
Member

Ok, looks like a regression from a couple weeks ago

  • Brave/OSX: can't stay logged in, fails to load Momentum (first screen)
  • FF/OSX: stays logged in, can't load Momentuym (second screen)

image

image

@pablo-mayrgundter pablo-mayrgundter merged commit c9e7d80 into bldrs-ai:main Feb 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pablo-mayrgundter pablo-mayrgundter Awaiting requested review from pablo-mayrgundter

@OlegMoshkovich OlegMoshkovich Awaiting requested review from OlegMoshkovich

Assignees
No one assigned
Labels
None yet
Projects
No open projects
bldrs.ai
Status: Completed
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@oo-bldrs @MarkusSteinbrecher @pablo-mayrgundter @OlegMoshkovich