Skip to content

feat: run catalogue and emx2 together on one domain by mixing their paths #5231

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

Open
wants to merge 37 commits into
base: master
Choose a base branch
from

Conversation

mswertz
Copy link
Member

@mswertz mswertz commented Jul 19, 2025

description:

  • paths that match an app are sent to emx2 platform, e.g. /someschema/tables
  • all other paths are sent to catalogue app
  • we added the /schema/index to do the magic of routing to the default app of another schema (e.g. in breadcrumb)

@mswertz mswertz changed the title feat: run catalogue and emx2 together without using /cms/ feat: run catalogue and emx2 together on one domain by mixing their paths Jul 20, 2025
Copy link

@mswertz mswertz requested a review from YpeZ July 20, 2025 09:48
Copy link
Contributor

@BrendaHijmans BrendaHijmans left a comment

Choose a reason for hiding this comment

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

When I am in a schema, such as: https://preview-catalogue-pr-5231.dev.molgenis.org/catalogue-demo/tables#/ and click on the dropdown (see picture) and click on the schema name (see picture), I get a 500 error

image

@BrendaHijmans
Copy link
Contributor

BrendaHijmans commented Jul 21, 2025

Something I noticed, don't know whether it is an issue. This url (with .../tables#/) is working, but used to redirect to ../tables/#/ and does not anymore. https://preview-catalogue-pr-5231.dev.molgenis.org/catalogue-demo/tables#/ does not redirect to https://preview-catalogue-pr-5231.dev.molgenis.org/catalogue-demo/tables/#/

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