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

Final Steps for Migrating Passport App to passport.xyz #2767

Closed
6 tasks
erichfi opened this issue Aug 7, 2024 · 3 comments · Fixed by #2833 or #2846
Closed
6 tasks

Final Steps for Migrating Passport App to passport.xyz #2767

erichfi opened this issue Aug 7, 2024 · 3 comments · Fixed by #2833 or #2846
Assignees

Comments

@erichfi
Copy link
Collaborator

erichfi commented Aug 7, 2024

Description:
We need to complete the following steps for the migration of the Passport app to passport.xyz:

  1. Update OAuth Callbacks:
  • Review the OAuth Callbacks listed in this document.
  • Identify missing credentials and create new OAuth apps where necessary.
  1. Redirect Domain: (We will not do as part of this work, we'll be running both in parallel for some time before we do the permanent redirect)
  • Redirect from passport.gitcoin.co to passport.xyz.
  • Access to the Gitcoin AWS account is required for this task.

Tasks:

  • Ping Gerald to request OAuth access.
  • Confirm OAuth access with either Gerald or Kyle.
  • Reach out to Aditya on the Gitcoin team regarding AWS account access.
  • Update OAuth Callbacks as per the document.
  • Implement domain redirection.
  • Migrate the IAM service to passport.xyz

Related Issue: GitHub Issue #2641

@erichfi erichfi moved this to Prioritized in Passport New Aug 7, 2024
@larisa17 larisa17 self-assigned this Aug 8, 2024
@Jkd-eth
Copy link
Contributor

Jkd-eth commented Aug 20, 2024

@Kweiss We'll need to get your support to get access to all of the Oauth applications

@larisa17
Copy link
Collaborator

larisa17 commented Sep 2, 2024

Status update :
Github successfully tested on review with the new app created in the passport.xyz organization
Google successfully tested on review with the new app created in the passport.xyz organization
Outdid: successfully tested on review with redirect URL changed from env variable
Civic: successfully tested on review on the new domain

WIP: linkedin @larisa17

To Be Done : Discord : new OAuth application is required

Still to be clarified Coinbase & Idena

@nutrina
Copy link
Collaborator

nutrina commented Sep 2, 2024

I am on to Coinbase

@larisa17 larisa17 linked a pull request Sep 4, 2024 that will close this issue
@Jkd-eth Jkd-eth moved this from Ready to Deploy to Done in Passport New Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
4 participants