Skip to content

PWA-277 FIX: update deploy workflow to support multiple apps - #7

Merged
larumanaulo merged 0 commit into
masterfrom
feature/PWA-277-custom-contentful-app-rich-text-html-converter
May 27, 2026
Merged

PWA-277 FIX: update deploy workflow to support multiple apps#7
larumanaulo merged 0 commit into
masterfrom
feature/PWA-277-custom-contentful-app-rich-text-html-converter

Conversation

@snowdoniadarkness

Copy link
Copy Markdown
Contributor

Purpose

What the new workflow does:

detect job: on push, uses git diff to find which apps changed and builds a matrix of only those apps. On workflow_dispatch, deploys the manually selected app.
deploy job: runs in parallel per-app using the matrix, each resolving its own CONTENTFUL_APP_DEF_ID_* secret dynamically via secrets[matrix.app_def_id_secret].
Adding a new app later: add one line to the APPS map in the detect step.

Action required — GitHub Secrets:

Rename/create these secrets in your repo settings (Settings → Secrets → Actions):
CONTENTFUL_APP_DEF_ID -> CONTENTFUL_APP_DEF_ID_RICH_TEXT_HTML_CONVERTER
(second app needs its own) -> CONTENTFUL_APP_DEF_ID_SALESFORCE_COMMERCE_CLOUD

CONTENTFUL_ORG_ID and CONTENTFUL_ACCESS_TOKEN stay shared across all apps.

Approach

Testing steps

Dependencies and/or References

Deployment

Comment thread .github/workflows/deploy-apps.yml Outdated
Comment on lines 21 to 62
Comment thread .github/workflows/deploy-apps.yml Outdated
Comment on lines 63 to 90
Comment thread .github/workflows/deploy-apps.yml Outdated
@larumanaulo
larumanaulo merged this pull request into master May 27, 2026
3 of 4 checks passed
snowdoniadarkness pushed a commit that referenced this pull request Jul 7, 2026
…-contentful-app-rich-text-html-converter

PWA-277 FIX: update deploy workflow to support multiple apps
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