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 shims for atob #88

Merged

Conversation

Ethella
Copy link
Member

@Ethella Ethella commented Apr 15, 2022

📦 Pull Request

Add shims for atob

✅ Fixed Issues

Fixes #86

🚨 Test instructions

[Describe any additional context required to test the PR/feature/bug fix.]

⚠️ Don't forget to add a semver label!

  • patch: Bug Fix?
  • minor: New Feature?
  • major: Breaking Change?
  • skip-release: It's unnecessary to publish this change.

@Ethella Ethella added the patch Increment the patch version when merged label Apr 15, 2022
@Ethella Ethella self-assigned this Apr 15, 2022
@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #50572: Admin-js Malformed Token on deployment.

import atob from 'atob';

// Shims for atob being undefined in node.js prior version 14
if (!globalThis.atob) globalThis.atob = atob;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Ethella Looks good to me. Just dropped the test coverage too low 🤣

Copy link
Member Author

@Ethella Ethella Apr 15, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wasn't aware of tests 😭. Will make them

dgerrellsMagic
dgerrellsMagic previously approved these changes Apr 15, 2022
@Ethella Ethella merged commit ffbff87 into master Apr 16, 2022
@Ethella Ethella deleted the jerryliu-sc-50572-admin-js-malformed-token-on-deployment branch April 16, 2022 15:55
@magiclabsFE
Copy link

🚀 PR was released in v1.4.1 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Malformed Token on deployment
4 participants