commits
=======
- build(renovate): automerge devDependencies
- build(renovate): use preset to schedule weekly
- build(renovate): use preset to rebaseStalePrs
Background
==========
As part of our Squad-based ownership, we should stay on top of what
happens in these repositories. However, due to the number of
repositories (and subsequently pull requests) across the edX ecosystem,
it is challenging to stay on top of notifications, separating the
'signal' from the 'noise'. Email filters can go a long way to taming
Inbox notifications, but this is manual and requires maintenance as
Squad ownership changes. It also fails to account for Github-specific behavior.
Proposal
========
By leveraging Github support for `CODEOWNERS` files [1],
we can ensure that our team is at least CCed explicitly, here,
in the form a requested review. This request is just that, a request,
not a requirement; we are not enacting any new merge requirements
at this time.
- [1] https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/about-code-owners
* build: defaulting coaching and demographics to false
Given that coaching and demographics are private, internal features to edx.org, we default them to false.
* docs: documenting custom MFE environment variables
Also adjusting the group to tag while I’m in here, and linking directly to the common env variables in read the docs.
* docs: improve formatting and fix sentence fragment in SUPPORT_URL
This repo was allowing both patch/minor updates of Paragon, but I've opened that up a bit with this PR. This is based on similar config we've used in other repositories.
It loses the "stability days" that were here, but I don't think we need that for patch/minor updates.
* fix: removing hard-coded edX from DemographicsSection messages
* fix: removing hard-coded “edX” strings from IDV messages
* fix: updating ThirdPartyAuth message description to remove hard-coded “edX” references.
* fix: replacing hard-coded “edX” strings with SITE_NAME in AccountSettingsPage
* fix: conditionalizing edx-specific strings in ConfirmationModal
If the SITE_NAME is ‘edX’, then edx.org-specific strings will be used. Otherwise, more general, Open edX-appropriate strings will be used.
* fix: conditionalizing edX-specific strings in delete account components.
If the SITE_NAME is ‘edX’, then edx.org-specific strings will be used. Otherwise, more general, Open edX-appropriate strings will be used.
* fix: replacing hard-coded ‘edX’ strings with SITE_NAME in ReviewRequirementsPanel
I missed a few because the messages were re-used.
* fix: review feedback, improving messages
- Removing unnecessary {siteName} references
- Improving some message descriptions
This uses variable substitution to insert SITE_NAME into index.html, rather than hard-coding “edX” into the file. It also updates the .env files to use “localhost” as the default SITE_NAME.
The new reroutes introduced in the current A/B test override the
summary panel reroute, resulting in incorrect behavior. This fix
ensures that, if a user has reached the summary panel once,
they are properly rerouted from the camera/upload panels when
going back to retake or reupload their photos.