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

front: NavBar: display license attributions #3551

Merged
merged 1 commit into from
Mar 16, 2023
Merged

Conversation

Yohh
Copy link
Contributor

@Yohh Yohh commented Mar 15, 2023

close #3255

@Yohh Yohh requested a review from a team as a code owner March 15, 2023 12:44
@codecov
Copy link

codecov bot commented Mar 15, 2023

Codecov Report

Merging #3551 (e2795f2) into dev (8d8d3d0) will increase coverage by 0.07%.
The diff coverage is n/a.

❗ Current head e2795f2 differs from pull request most recent head f64e48d. Consider uploading reports for the commit f64e48d to get more accurate results

@@             Coverage Diff              @@
##                dev    #3551      +/-   ##
============================================
+ Coverage     68.07%   68.14%   +0.07%     
  Complexity     1897     1897              
============================================
  Files           420      417       -3     
  Lines         20983    20949      -34     
  Branches       1534     1534              
============================================
- Hits          14284    14276       -8     
+ Misses         5984     5958      -26     
  Partials        715      715              
Flag Coverage Δ
front 56.50% <ø> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 13 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Yohh Yohh force-pushed the yoh/addLicenseAttributions branch 3 times, most recently from a06cc76 to e0d2347 Compare March 15, 2023 14:16
front/public/locales/fr/translation.json Outdated Show resolved Hide resolved
@Yohh Yohh force-pushed the yoh/addLicenseAttributions branch from e0d2347 to 12368be Compare March 16, 2023 09:52
Copy link
Contributor

@clarani clarani left a comment

Choose a reason for hiding this comment

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

Nice job ! I left some comments

front/public/locales/fr/translation.json Outdated Show resolved Hide resolved
front/src/common/ReleaseInformations/consts.ts Outdated Show resolved Hide resolved
front/src/common/ReleaseInformations/consts.ts Outdated Show resolved Hide resolved
@Yohh Yohh force-pushed the yoh/addLicenseAttributions branch 4 times, most recently from a370b24 to f7cd37d Compare March 16, 2023 14:40
…collect attributions then add it to a const in consts.ts; edit translation.json

- set LicenseAttributions in ReleaseInformations; change informations button to about, add translation

- edit LicenseAttribution logic, remove License interface and informations translation

- remove consts.ts, import licenses.json in LicenseAttributions.tsx, add script to package.json, update front readme
@Yohh Yohh force-pushed the yoh/addLicenseAttributions branch from f7cd37d to f64e48d Compare March 16, 2023 15:58
Copy link
Contributor

@clarani clarani left a comment

Choose a reason for hiding this comment

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

LGTM !! 🥳

@Yohh Yohh requested a review from multun March 16, 2023 16:17
@Yohh Yohh merged commit cffce98 into dev Mar 16, 2023
@Yohh Yohh deleted the yoh/addLicenseAttributions branch March 16, 2023 16:19
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.

Properly credit front-end libs
3 participants