-
Notifications
You must be signed in to change notification settings - Fork 116
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 a bug with relational mapping tests set with the incorrect database #645
Conversation
🦋 Changeset detectedLatest commit: eb3a1ab The changes in this PR will be included in the next version bump. This PR includes changesets to release 13 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Codecov Report
@@ Coverage Diff @@
## master #645 +/- ##
==========================================
- Coverage 43.64% 43.63% -0.02%
==========================================
Files 904 904
Lines 40301 40309 +8
Branches 9238 9241 +3
==========================================
- Hits 17589 17588 -1
- Misses 22648 22657 +9
Partials 64 64
|
@YannanGao-gs Could you elaborate on the context of this bug, what are you aiming to fix? |
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.
Very minor comments. Good to approve once the changeset is addressed. Thanks!
packages/legend-studio/src/stores/editor-state/element-editor-state/mapping/MappingTestState.ts
Outdated
Show resolved
Hide resolved
...components/editor/edit-panel/mapping-editor/InstanceSetImplementationSourceSelectorModal.tsx
Show resolved
Hide resolved
a7abcd6
to
f18a060
Compare
f18a060
to
2f139d4
Compare
Summary
Fixes #649
How did you test this change?
demo.mov