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

publish doc website and improve Legend Studio analytics reporting #1266

Merged
merged 18 commits into from
Jul 10, 2022

Conversation

akphi
Copy link
Contributor

@akphi akphi commented Jul 6, 2022

Summary

  • publish doc website
  • bump dependencies
  • Fix styling problem for duplicated projection name in query builder
  • Send analytics about users' application navigation context:
    • text mode editor
    • model loader
    • studio embedded query builder
    • most of studio editors: class, profile, enumeration, mapping, etc.
  • Send analytics about virtual assistant interactivity:
    • when users open virtual assitant
    • when users click on documentation key
  • Cleanup post Support property search in query builder #1212
    • Only show drag-layer when dragging found property nodes
    • Simplify drag/drop event hook usage
    • Make search panel smaller
    • Make search property button a toggler
    • Auto-focus on the search field when search panel is open
    • Minor layout re-adjustment
  • Run build doc as part of codebase check CI
  • Minor refactor to facilitate Feature request: Improve UX when loading dataspaces #936

How did you test this change?

  • Test(s) added
  • Manual testing (please provide screenshots/recordings)
  • No testing (please provide an explanation)

@changeset-bot
Copy link

changeset-bot bot commented Jul 6, 2022

🦋 Changeset detected

Latest commit: f52780b

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

This PR includes changesets to release 31 packages
Name Type
@finos/legend-application Major
@finos/legend-studio Major
@finos/legend-art Patch
@finos/legend-extension-dsl-data-space Patch
@finos/legend-extension-dsl-diagram Patch
@finos/legend-extension-dsl-persistence Patch
@finos/legend-extension-dsl-text Patch
@finos/legend-extension-external-store-service Patch
@finos/legend-graph Minor
@finos/legend-query Patch
@finos/legend-query-app Patch
@finos/legend-studio-app Patch
@finos/legend-studio-extension-management-toolkit Patch
@finos/legend-studio-extension-query-builder Patch
@finos/legend-taxonomy Patch
@finos/legend-taxonomy-app Patch
@finos/legend-dev-utils Patch
@finos/legend-model-storage Patch
@finos/legend-server-depot Patch
@finos/legend-server-sdlc Patch
@finos/legend-shared Patch
@finos/stylelint-config-legend-studio Patch
@finos/legend-extension-external-language-morphir Patch
@finos/legend-extension-mapping-generation Patch
@finos/legend-manual-tests Patch
@finos/legend-graph-extension-collection Patch
@finos/legend-extension-external-format-json-schema Patch
@finos/legend-query-deployment Patch
@finos/legend-studio-deployment Patch
@finos/legend-taxonomy-deployment Patch
@finos/legend-tracer-extension-zipkin 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

@finos-cla-bot finos-cla-bot bot added the cla-present CLA Signed label Jul 6, 2022
@akphi akphi force-pushed the caoi branch 2 times, most recently from 4ab079e to 7fd0431 Compare July 6, 2022 18:56
@codecov
Copy link

codecov bot commented Jul 6, 2022

Codecov Report

Merging #1266 (f52780b) into master (9a8b095) will increase coverage by 0.01%.
The diff coverage is 36.51%.

@@            Coverage Diff             @@
##           master    #1266      +/-   ##
==========================================
+ Coverage   41.71%   41.72%   +0.01%     
==========================================
  Files        1143     1147       +4     
  Lines       51976    52051      +75     
  Branches    11819    11829      +10     
==========================================
+ Hits        21681    21719      +38     
- Misses      30228    30265      +37     
  Partials       67       67              
Impacted Files Coverage Δ
...end-application/src/stores/ApplicationTelemetry.ts 0.00% <0.00%> (ø)
...on/src/stores/LegendApplicationAssistantService.ts 10.44% <0.00%> (ø)
packages/legend-art/src/components/Input.tsx 0.00% <ø> (ø)
packages/legend-art/src/components/LegendLogo.tsx 0.00% <ø> (ø)
...kages/legend-art/src/components/ResizablePanel.tsx 10.71% <ø> (ø)
packages/legend-art/src/components/TreeView.tsx 9.09% <ø> (ø)
...nd-art/src/components/dialog/NonBlockingDialog.tsx 17.85% <ø> (ø)
...art/src/components/markdown/MarkdownTextViewer.tsx 0.00% <ø> (ø)
...ges/legend-art/src/components/menu/MenuContent.tsx 0.00% <ø> (ø)
...end-art/src/components/panel/BlankPanelContent.tsx 0.00% <ø> (ø)
... and 63 more

@akphi akphi mentioned this pull request Jul 7, 2022
20 tasks
@akphi akphi force-pushed the caoi branch 6 times, most recently from a772f41 to c987dc6 Compare July 8, 2022 16:38
@akphi akphi marked this pull request as ready for review July 10, 2022 05:34
@akphi akphi changed the title improve dataspace performance by using analytics API publish doc website and improve Legend Studio analytics reporting Jul 10, 2022
@akphi akphi merged commit 7a967b8 into finos:master Jul 10, 2022
@akphi akphi deleted the caoi branch July 10, 2022 05:35
jinanisha pushed a commit to jinanisha/legend-studio that referenced this pull request Aug 5, 2022
…inos#1266)

* give more usage analytics

* fix problem with input with context doc do not properly receive focus

* bump dependencies

* minor fixes

* query builder: minor style fix for duplicated properties for projections

* introduce extension mechanism for AbstractPureGraphManager

* cleanup post finos#1212

* send analytics for more editors

* minor fixes to cleanup test warnings

* minor fix

* bump dependencies

* update typedoc setup

* remove project coordinate from dataspace tagged value for query

* bump dependencies

* minor cleanups

* reorganize Legend Query code

* update typedoc version

* publish doc website
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-present CLA Signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant