-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: patch to production app for dev projects page - DO NOT MERGE #4689
base: dev
Are you sure you want to change the base?
Conversation
@@ -13,15 +13,15 @@ | |||
heading="Simulation projects" | |||
icon="project" | |||
id="trigger" | |||
[routerLink]="['/projects']" | |||
[href]="config.platformAppProjectsUrl" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's it? hope it works!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, the code change is easy - the corresponding monkey-patch would have been brutal if I could not rebuild the frontend.
DO NOT MERGE - will be pushed to Netlify only |
Oh man, this seems fun. How are we rebuilding the prod site to do the patch? |
Expanding the scope of the production site patches to bypass the biosimulators.org landing page (redirect to /simulators to directly display the list of simulators) - see #4706. |
SonarCloud Quality Gate failed. 0 Bugs No Coverage information |
What new features does this PR implement?
patch production site (v9.13.6) to link out to https://biosimulations.dev/projects for all project browsing, and to https://explorersb-test.pages.dev for a preview of the new simplified Reproducibility Portal. see Issue #4688.
(added 6/14/2023) - patch production site to bypass biosimulators.org landing page, redirecting to the list of simulators at the /simulators path. This is to avoid confusing navigation while transitioning to the new navigation scheme in the new development versions. see Issue #4706
What bugs does this PR fix?
Motivation is to highlight recent development on project browsing without touching the production site (because dev site is not yet ready to replace the production site).
How have you tested this PR?
build/tested locally (localhost:4200)
will upload build directly to Netlify for only platform app at biosimulations.org - to bypass CI/CD (broken for 9.13.6)