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

Add getCoreApiBaseEndpoint helper #88

Merged
merged 4 commits into from
Sep 10, 2024
Merged

Add getCoreApiBaseEndpoint helper #88

merged 4 commits into from
Sep 10, 2024

Conversation

marcomontalbano
Copy link
Member

@marcomontalbano marcomontalbano commented Sep 3, 2024

Linked to commercelayer/issues-app#182

What I did

I created a new helper method called getCoreApiBaseEndpoint that can derive the Core API base endpoint given a valid access token.

How to test

getCoreApiBaseEndpoint('eyJhbGciOiJS...') //= "https://yourdomain.commercelayer.io"

Checklist

  • Make sure your changes are tested (stories and/or unit, integration, or end-to-end tests).
  • Make sure to add/update documentation regarding your changes.
  • You are NOT deprecating/removing a feature.

Copy link

vercel bot commented Sep 3, 2024

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

Name Status Preview Comments Updated (UTC)
commercelayer-js-auth-nextjs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 10, 2024 2:18pm

@marcomontalbano marcomontalbano self-assigned this Sep 3, 2024
@marcomontalbano marcomontalbano added the enhancement New feature or request label Sep 3, 2024
Copy link

netlify bot commented Sep 3, 2024

Deploy Preview for commercelayer-js-auth-nextjs ready!

Name Link
🔨 Latest commit 321ef55
🔍 Latest deploy log https://app.netlify.com/sites/commercelayer-js-auth-nextjs/deploys/66e054f0bba3ce0008a027e9
😎 Deploy Preview https://deploy-preview-88--commercelayer-js-auth-nextjs.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 configuration.

gciotola
gciotola previously approved these changes Sep 4, 2024
@malessani malessani merged commit ca1e628 into main Sep 10, 2024
6 checks passed
@malessani malessani deleted the get-core-api-endpoint branch September 10, 2024 15:59
@marcomontalbano marcomontalbano changed the title Add getCoreApiEndpoint helper Add getCoreApiBaseEndpoint helper Sep 10, 2024
@marcomontalbano marcomontalbano changed the title Add getCoreApiBaseEndpoint helper Add getCoreApiBaseEndpoint helper Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants