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

fix: Use header to get the client id #547

Merged
merged 31 commits into from
Mar 16, 2023
Merged

Conversation

Jordan-Hall
Copy link
Contributor

@Jordan-Hall Jordan-Hall commented Mar 8, 2023

ref: #427

@vercel
Copy link

vercel bot commented Mar 8, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
fitlink-monorepo ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 16, 2023 at 2:52PM (UTC)

@Jordan-Hall
Copy link
Contributor Author

@paulgosnell we will need to add the Environment vars for the build to work correctly in the secrets

FITLINK Strava

  FITLINK_STRAVA_CLIENT_ID: ${{ secrets.DEV_FITLINK_STRAVA_CLIENT_ID }}
  FITLINK_STRAVA_CLIENT_SECRET: ${{ secrets.DEV_FITLINK_STRAVA_CLIENT_SECRET }}

  # FITLINK Fitbit
  FITLINK_FITBIT_CLIENT_ID: ${{ secrets.DEV_FITLINK_FITBIT_CLIENT_ID }}
  FITLINK_FITBIT_CLIENT_SECRET: ${{ secrets.DEV_FITLINK_FITBIT_CLIENT_SECRET }}
  FITLINK_FITBIT_VERIFY_WEBHOOK_DEFAULT: ${{ secrets.DEV_FITLINK_FITBIT_VERIFY_WEBHOOK_DEFAULT }}
  FITLINK_FITBIT_VERIFY_WEBHOOK_ACTIVITIES: ${{ secrets.DEV_FITLINK_FITBIT_VERIFY_WEBHOOK_ACTIVITIES }}
  FITLINK_FITBIT_VERIFY_WEBHOOK_SLEEP: ${{ secrets.DEV_FITLINK_FITBIT_VERIFY_WEBHOOK_SLEEP }}

  # BFIT Strava
  BFIT_STRAVA_CLIENT_ID: ${{ secrets.DEV_BFIT_STRAVA_CLIENT_ID }}
  BFIT_STRAVA_CLIENT_SECRET: ${{ secrets.DEV_BFIT_STRAVA_CLIENT_SECRET }}

  # BFIT Fitbit
  BFIT_FITBIT_CLIENT_ID: ${{ secrets.DEV_BFIT_FITBIT_CLIENT_ID }}
  BFIT_FITBIT_CLIENT_SECRET: ${{ secrets.DEV_BFIT_FITBIT_CLIENT_SECRET }}
  BFIT_FITBIT_VERIFY_WEBHOOK_DEFAULT: ${{ secrets.DEV_BFIT_FITBIT_VERIFY_WEBHOOK_DEFAULT }}
  BFIT_FITBIT_VERIFY_WEBHOOK_ACTIVITIES: ${{ secrets.DEV_BFIT_FITBIT_VERIFY_WEBHOOK_ACTIVITIES }}
  BFIT_FITBIT_VERIFY_WEBHOOK_SLEEP: ${{ secrets.DEV_BFIT_FITBIT_VERIFY_WEBHOOK_SLEEP }}

@paulgosnell
Copy link
Contributor

FITLINK_FITBIT_CLIENT_ID

This is done @Jordan-Hall - all added to both AWS EB env vars on prod and Github secrets

added a default to the X-Client-ID for if its not set it goes to FitLink this is due to not updating fitlink

Signed-off-by: Jordan Hall <[email protected]>
Signed-off-by: Jordan Hall <[email protected]>
Signed-off-by: Jordan Hall <[email protected]>
Signed-off-by: Jordan Hall <[email protected]>
Signed-off-by: Jordan Hall <[email protected]>
@Jordan-Hall Jordan-Hall force-pushed the feat/427-bunld-ip-driven branch from 0a43a94 to 424e333 Compare March 12, 2023 22:23
Signed-off-by: Jordan Hall <[email protected]>
Signed-off-by: Jordan Hall <[email protected]>
Signed-off-by: Jordan Hall <[email protected]>
Signed-off-by: Jordan Hall <[email protected]>
@Jordan-Hall Jordan-Hall marked this pull request as ready for review March 16, 2023 15:01
@Jordan-Hall Jordan-Hall merged commit 7e0f1e4 into develop Mar 16, 2023
@Jordan-Hall Jordan-Hall deleted the feat/427-bunld-ip-driven branch March 16, 2023 15:13
Jordan-Hall added a commit that referenced this pull request Mar 16, 2023
* fix: Use header to get the client id

ref: #427
Jordan-Hall added a commit that referenced this pull request Mar 20, 2023
* ✨ 548 - auth logic base (Enter + Create + cooldown + on active)

* fix: Use header to get the client id (#547)

* fix: Use header to get the client id

ref: #427

* ✨ 548 - final auth version

* ✨ 548 - add setting toggle, request biometry on active

* ✨ 548 - podfile

* removed dollar item

* fixes

* fix: repair strava (#593)

* fix: repair strava

Signed-off-by: Jordan Hall <[email protected]>

* fix: repair unit test

Signed-off-by: Jordan Hall <[email protected]>

---------

Signed-off-by: Jordan Hall <[email protected]>

---------

Signed-off-by: Jordan Hall <[email protected]>
Co-authored-by: KirillRodichev <[email protected]>
Co-authored-by: Kirill Rodichev <[email protected]>
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.

3 participants