Skip to content
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

feat: ✨ Allow bundle name spacing #1774

Merged
merged 3 commits into from
Jan 4, 2021

Conversation

ashuverma
Copy link
Contributor

@ashuverma ashuverma commented Jan 4, 2021

  • Currently everything gets read from bundleAssets and put it under window.xarcv1.rt.bundles directly.
  • This PR adds name-spacing capabilities to have multiple bundle assets and allows subapps to be loaded from different namespaces.
  • backward compatible. If there is no namespace provided, defaults to root.

cc @jchip @divyakarippath

jchip and others added 2 commits January 3, 2021 00:05
Currently everything gets read from bundleAssets and put it under window.xarcv1.rt.bundles directly. This PR adds namespacing capabilities to have multiple bundle assets and allows subapps to be loaded from different namespaces.
@jchip jchip force-pushed the master branch 2 times, most recently from eddd345 to 27f6236 Compare January 4, 2021 17:29
@jchip jchip merged commit 2d56858 into electrode-io:master Jan 4, 2021
@ashuverma ashuverma deleted the add-bundle-namespacing branch January 5, 2021 04:45
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