Skip to content

PJH-146: Use SHOPIFY_CLI_PARTNERS_TOKEN for Shopify deploy#143

Open
kai-nguyen-aligent wants to merge 1 commit intomainfrom
fix/PJH-146-update-shopify-workflow
Open

PJH-146: Use SHOPIFY_CLI_PARTNERS_TOKEN for Shopify deploy#143
kai-nguyen-aligent wants to merge 1 commit intomainfrom
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.

working-directory: ${{ inputs.working-directory }}
run: yarn shopify app deploy --allow-updates
env:
SHOPIFY_CLI_TOKEN: ${{ secrets.shopify_cli_token }}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there harm in having both? My concern is when a client moves off our account, they can't use our partner token anymore. If this workflow only supports partner tokens deploys will break

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