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

fix: init multiple Application instances while starting up a sub app #1986

Merged
merged 6 commits into from
Jun 6, 2023

Conversation

icezohu
Copy link
Contributor

@icezohu icezohu commented Jun 5, 2023

starting up sub app

by Jacob

Note
This is a template for submitting a new feature. Use the bug fix template if you're submitting a bug fix pull request by adding template=bug_fix.md to your pull request URL.

Description (需求描述)

Describe the new feature or modification to an existing feature clearly and consciously.

Motivation (需求背景)

Explain the reason for adding or modifying this feature.

Key changes (关键改动)

Provide a technically detailed description of the key changes made.

  • Frontend (前端)
  • Backend (后端)

Test plan (测试计划)

Suggestions (测试建议)

Provide any suggestions or recommendations for improvements in the testing plan.

Underlying risk (潜在风险)

Identify any potential risks or issues that may arise from the new feature or modification.

Showcase (结果展示)

Including any screenshots of the new feature or modification.

@height
Copy link

height bot commented Jun 5, 2023

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

when asynchronously access with same sub app name

by Jacob
to multi-app-manager package.json

by Jacob
@chareice chareice merged commit bc00bd1 into nocobase:main Jun 6, 2023
17 checks passed
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.

None yet

2 participants