Skip to content

[ACT-900] Fix unsupported tours - #137

Closed
davidnuon wants to merge 4 commits into
openedx:mainfrom
edx:act-900/fix-unsupported-tours
Closed

davidnuon wants to merge 4 commits into
openedx:mainfrom
edx:act-900/fix-unsupported-tours

Conversation

@davidnuon

Copy link
Copy Markdown

Description

Ticket: https://2u-internal.atlassian.net/browse/ACT-900

How Has This Been Tested?

Please describe in detail how you tested your changes.

Screenshots/sandbox (optional):

Include a link to the sandbox for design changes or screenshot for before and after. Remove this section if it's not applicable.

Before After

Merge Checklist

  • If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Sandbox, if applicable.
  • Is there adequate test coverage for your changes?

Post-merge Checklist

  • Deploy the changes to prod after verifying on stage or ask @openedx/edx-infinity to do it.
  • 🎉 🙌 Celebrate! Thanks for your contribution.

* chore: publish from the edx fork under the @edx scope

- .releaserc: replace upstream's stable/N.x branch set with a single
  `release` branch on the `latest` channel; `main` stays alpha.
- release.yml: trigger on main and release, and release with the edX
  org's SEMANTIC_RELEASE_NPM_TOKEN plus the default GITHUB_TOKEN
  instead of OPENEDX_SEMANTIC_RELEASE_GITHUB_TOKEN. Drop id-token,
  since npm provenance is unavailable for restricted packages.
- package.json: rename to @edx/frontend-app-notifications, publish
  restricted, and repoint repository/homepage/bugs at the fork.
- README: fork identity, badges, and install/import instructions.

* chore: run Default CI on the release branch instead of upstream branches
The discussion tours API can return tours this app has no checkpoints
for. These were mapped to `false`, which crashed Paragon's ProductTour.
Filter them out, persist dismissals via onDismiss, and use
dismissAltText in place of the deprecated dismissButtonText.
@davidnuon davidnuon closed this Sep 30, 2026
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.

1 participant