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: Show code snipped in diagram cache dialog #1147

Merged
merged 3 commits into from
Dec 15, 2023
Merged

Conversation

Paula-Kli
Copy link
Contributor

In order for users to better understand how a personal access token can be helpful the codesnippet from the docs is displayed in the frontend.

closes #598

@codecov
Copy link

codecov bot commented Oct 30, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (f6e8fb9) 72.69% compared to head (89b8ebf) 72.72%.
Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1147      +/-   ##
==========================================
+ Coverage   72.69%   72.72%   +0.02%     
==========================================
  Files         162      162              
  Lines        5315     5320       +5     
  Branches      597      597              
==========================================
+ Hits         3864     3869       +5     
  Misses       1320     1320              
  Partials      131      131              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Paula-Kli
Copy link
Contributor Author

Currently looks like
Bildschirmfoto 2023-10-31 um 11 44 40

I used CSS in order to style it as I thought would be nicer than javascript - nevertheless the copied code still has a lot of white spaces in front even though they do not get displayed...

@Paula-Kli Paula-Kli marked this pull request as ready for review October 31, 2023 10:49
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

In order for users to better understand how a personal access token can be helpful the codesnippet from the docs is displayed in the frontend.
@MoritzWeber0 MoritzWeber0 force-pushed the show-codesnippet branch 2 times, most recently from c3d3fb2 to 4d390cf Compare December 14, 2023 18:55
Copy link
Contributor

@dominik003 dominik003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! (Only one important thing to adjust, i.e., properly handle the error case in the model ... dialog)

@MoritzWeber0 MoritzWeber0 force-pushed the show-codesnippet branch 3 times, most recently from 88943d3 to 24bdf70 Compare December 15, 2023 11:04
The commits identifies the primary Git model of a model and
adds the information to the code snippet of the diagram cache.

In addition, there were some minor/internal changes:

- The code snippet code was moved to a new component `ModelDiagramCacheBlockComponent`.
- Syntax highlighting was added in the frontend.
- A duplication of the code snippet declaration was removed.
- Margins and paddings were updated, responsive design was added.
- The colors of the code snippet were inverted (white text on dark background)
- Information about token revocation was added to the dialog.
- A confirmation message for clipboard-copy was added.
Copy link

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

1 New issue
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@MoritzWeber0 MoritzWeber0 merged commit 84bc4e6 into main Dec 15, 2023
24 checks passed
@MoritzWeber0 MoritzWeber0 deleted the show-codesnippet branch December 15, 2023 14:04
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.

Show code snippet and generate techusers for diagram cache
3 participants