finos / legend-studio

Legend Studio
https://legend.finos.org
Apache License 2.0
87 stars 113 forks source link

Improve Help button in Query Builder #3257

Closed YannanGao-gs closed 2 months ago

YannanGao-gs commented 2 months ago

Summary

Improve Help button in Query Builder

How did you test this change?

https://github.com/finos/legend-studio/assets/73408381/59df2e3c-e93e-4eb0-989a-92b5e336e980

Screenshot 2024-06-17 at 10 09 35 AM Screenshot 2024-06-14 at 4 11 46 PM
changeset-bot[bot] commented 2 months ago

🦋 Changeset detected

Latest commit: 695a268c08cbfc161519142c41473b1a06b5c787

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 24 packages | Name | Type | | --------------------------------------------- | ----- | | @finos/legend-application-query | Patch | | @finos/legend-application-query-bootstrap | Patch | | @finos/legend-query-builder | Patch | | @finos/legend-application | Patch | | @finos/legend-extension-dsl-service | Patch | | @finos/legend-application-query-deployment | Patch | | @finos/legend-application-studio | Patch | | @finos/legend-extension-dsl-data-space-studio | Patch | | @finos/legend-extension-dsl-data-space | Patch | | @finos/legend-application-pure-ide-deployment | Patch | | @finos/legend-application-pure-ide | Patch | | @finos/legend-application-repl-deployment | Patch | | @finos/legend-application-repl | Patch | | @finos/legend-application-studio-bootstrap | Patch | | @finos/legend-extension-assortment | Patch | | @finos/legend-extension-dsl-diagram | Patch | | @finos/legend-extension-dsl-persistence | Patch | | @finos/legend-extension-dsl-text | Patch | | @finos/legend-extension-store-flat-data | Patch | | @finos/legend-extension-store-relational | Patch | | @finos/legend-extension-store-service-store | Patch | | @finos/legend-lego | Patch | | @finos/legend-application-studio-deployment | Patch | | @finos/legend-server-showcase-deployment | Patch |

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[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 36.46617% with 169 lines in your changes missing coverage. Please review.

Project coverage is 48.07%. Comparing base (dbba035) to head (695a268). Report is 6 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #3257 +/- ## ========================================= Coverage 48.07% 48.07% ========================================= Files 1997 1998 +1 Lines 338983 339420 +437 Branches 14942 10568 -4374 ========================================= + Hits 162961 163190 +229 - Misses 175213 175686 +473 + Partials 809 544 -265 ``` | [Files](https://app.codecov.io/gh/finos/legend-studio/pull/3257?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos) | Coverage Δ | | |---|---|---| | [...d-application-query/src/stores/QueryEditorStore.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3257?src=pr&el=tree&filepath=packages%2Flegend-application-query%2Fsrc%2Fstores%2FQueryEditorStore.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uLXF1ZXJ5L3NyYy9zdG9yZXMvUXVlcnlFZGl0b3JTdG9yZS50cw==) | `37.89% <100.00%> (-1.72%)` | :arrow_down: | | [...c/stores/data-space/DataSpaceQueryBuilderHelper.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3257?src=pr&el=tree&filepath=packages%2Flegend-application-query%2Fsrc%2Fstores%2Fdata-space%2FDataSpaceQueryBuilderHelper.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uLXF1ZXJ5L3NyYy9zdG9yZXMvZGF0YS1zcGFjZS9EYXRhU3BhY2VRdWVyeUJ1aWxkZXJIZWxwZXIudHM=) | `38.88% <100.00%> (+0.57%)` | :arrow_up: | | [...application/src/components/ReleaseNotesManager.tsx](https://app.codecov.io/gh/finos/legend-studio/pull/3257?src=pr&el=tree&filepath=packages%2Flegend-application%2Fsrc%2Fcomponents%2FReleaseNotesManager.tsx&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uL3NyYy9jb21wb25lbnRzL1JlbGVhc2VOb3Rlc01hbmFnZXIudHN4) | `0.00% <ø> (ø)` | | | [...y-builder/src/\_\_lib\_\_/QueryBuilderDocumentation.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3257?src=pr&el=tree&filepath=packages%2Flegend-query-builder%2Fsrc%2F__lib__%2FQueryBuilderDocumentation.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLXF1ZXJ5LWJ1aWxkZXIvc3JjL19fbGliX18vUXVlcnlCdWlsZGVyRG9jdW1lbnRhdGlvbi50cw==) | `100.00% <100.00%> (ø)` | | | [...d-application-query/src/components/QueryEditor.tsx](https://app.codecov.io/gh/finos/legend-studio/pull/3257?src=pr&el=tree&filepath=packages%2Flegend-application-query%2Fsrc%2Fcomponents%2FQueryEditor.tsx&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uLXF1ZXJ5L3NyYy9jb21wb25lbnRzL1F1ZXJ5RWRpdG9yLnRzeA==) | `57.88% <41.66%> (-0.25%)` | :arrow_down: | | [...c/components/Core\_LegendQueryApplicationPlugin.tsx](https://app.codecov.io/gh/finos/legend-studio/pull/3257?src=pr&el=tree&filepath=packages%2Flegend-application-query%2Fsrc%2Fcomponents%2FCore_LegendQueryApplicationPlugin.tsx&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uLXF1ZXJ5L3NyYy9jb21wb25lbnRzL0NvcmVfTGVnZW5kUXVlcnlBcHBsaWNhdGlvblBsdWdpbi50c3g=) | `49.92% <67.56%> (+0.97%)` | :arrow_up: | | [...gend-query-builder/src/components/QueryBuilder.tsx](https://app.codecov.io/gh/finos/legend-studio/pull/3257?src=pr&el=tree&filepath=packages%2Flegend-query-builder%2Fsrc%2Fcomponents%2FQueryBuilder.tsx&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLXF1ZXJ5LWJ1aWxkZXIvc3JjL2NvbXBvbmVudHMvUXVlcnlCdWlsZGVyLnRzeA==) | `76.13% <40.90%> (-1.44%)` | :arrow_down: | | [...-query/src/components/data-space/DataSpaceInfo.tsx](https://app.codecov.io/gh/finos/legend-studio/pull/3257?src=pr&el=tree&filepath=packages%2Flegend-application-query%2Fsrc%2Fcomponents%2Fdata-space%2FDataSpaceInfo.tsx&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uLXF1ZXJ5L3NyYy9jb21wb25lbnRzL2RhdGEtc3BhY2UvRGF0YVNwYWNlSW5mby50c3g=) | `23.92% <23.92%> (ø)` | | ... and [168 files with indirect coverage changes](https://app.codecov.io/gh/finos/legend-studio/pull/3257/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos)
MauricioUyaguari commented 2 months ago

In About DataSpace, can we add an entry for Project Project: artifactId:groupID:version (PROD-123)

MauricioUyaguari commented 2 months ago

lets make sure all links go to the studio sdlc links not the archieve ones