Skip to content

PJH-146: Use SHOPIFY_CLI_PARTNERS_TOKEN for Shopify deploy#143

Merged
kai-nguyen-aligent merged 1 commit into
mainfrom
fix/PJH-146-update-shopify-workflow
May 5, 2026
Merged

PJH-146: Use SHOPIFY_CLI_PARTNERS_TOKEN for Shopify deploy#143
kai-nguyen-aligent merged 1 commit into
mainfrom
fix/PJH-146-update-shopify-workflow

Conversation

@kai-nguyen-aligent
Copy link
Copy Markdown
Contributor

@kai-nguyen-aligent kai-nguyen-aligent commented Apr 30, 2026

Summary

  • Updates the Shopify deploy workflow to use SHOPIFY_CLI_PARTNERS_TOKEN instead of SHOPIFY_CLI_TOKEN as the environment variable for authentication
  • Updates the secret description to clarify it is a partner authentication token

For more information around this change: https://shopify.dev/docs/apps/launch/deployment/deploy-in-ci-cd-pipeline#step-3-integrate-shopify-cli-into-your-pipeline

Copy link
Copy Markdown
Member

@TheOrangePuff TheOrangePuff left a comment

Choose a reason for hiding this comment

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

Not sure about this one. What happens when the client moves off our partner account and into their own?

@kai-nguyen-aligent
Copy link
Copy Markdown
Contributor Author

@TheOrangePuff After reading these documents, I'm not even sure if the client can move off our partner account or not.

However, Claude tells me that:

Shopify CLI expects the token to be set as the environment variable SHOPIFY_CLI_PARTNERS_TOKEN.

Verify the reusable workflow (aligent/workflows/.github/workflows/shopify-deploy.yml) is exposing the token as the correct environment variable. The workflow should have something like:

env:
  SHOPIFY_CLI_PARTNERS_TOKEN: ${{ secrets.shopify_cli_token }}

If it's setting a different env var name (e.g., SHOPIFY_CLI_TOKEN), that won't work — it must be SHOPIFY_CLI_PARTNERS_TOKEN.

Comment thread .github/workflows/shopify-deploy.yml
@kai-nguyen-aligent kai-nguyen-aligent merged commit 21f60bd into main May 5, 2026
3 checks passed
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.

2 participants