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

fix: Squashed commits of various fixes #8408, #8332, #8354, #6525, #3372 #8437

Merged
merged 25 commits into from
Jun 8, 2023

Conversation

aadak99
Copy link
Contributor

@aadak99 aadak99 commented May 20, 2023

Squashed my commits together

#8408
Added YouTube link to producers menu for tutorials.
@alexgarel revised it and added a string so that the video tutorials could be translated.
***The only issue I ran into was, the link translation string needs to be added to common.po and common.pot, but I do not have permission to assist with that. It would need to be added for entry "video_tutorials"

#8332
Added producers link at the top nav bar.
238206863-ccc567ae-9e75-4403-a489-8f897b4b2c31

#8354
Added correct link under Italian translation.
f6d9d038765525ec6fa269747bc17ab1

#6525
Made fix to typo aspect to aspects.
e719bdabad6f25de6cde2c0170bc159a

#3372
Updated world string to not change to FR on main English page of openfoodfacts
6a3ae453810a40b4a166f7204daf846c

Fixes #8408, #8332, #8354, #6525, #3372

Sorry for duplicate entries, trying to clean things up a bit and get it organized. Should be better going forward!

@aadak99 aadak99 requested a review from a team as a code owner May 20, 2023 19:43
@github-actions github-actions bot added product history We have kept 10 years of product revisions. This is useful to monitor edits & product improvements Template::Toolkit The templating toolkit used by product opener. The starting point for HTML/JS/CSS fixes. labels May 20, 2023
@alexgarel
Copy link
Member

Hi @aadak99, thanks for this PR. Normally you should also have modifications to po/common.pot and to po/common/en.po to add your translations strings but I don't see them in the PR.

@aadak99
Copy link
Contributor Author

aadak99 commented May 23, 2023

Hi @aadak99, thanks for this PR. Normally you should also have modifications to po/common.pot and to po/common/en.po to add your translations strings but I don't see them in the PR.

I'll look into adding those now, didn't realize it had to be done through common.pot/.po. I thought it could be added directly into the translation website. Thanks for your help and patience, once I get familiar with more, I will be able to consistently help out.

@github-actions github-actions bot added the Translations We use a non-standard version of GetText, lack language variants support translate.openfoodfacts.org label May 23, 2023
Copy link
Contributor Author

@aadak99 aadak99 left a comment

Choose a reason for hiding this comment

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

Added translatable string for video_tutorials

@@ -216,6 +217,7 @@
<ul class="search_and_links">
<li><a href="[% lang('menu_discover_link') %]" class="top-bar-links">[% lang('menu_discover') %]</a></li>
<li><a href="[% lang('menu_contribute_link') %]" class="top-bar-links">[% lang('menu_contribute') %]</a></li>
<li><a href="[% lang('footer_producers_link') %]" class="top-bar-links">[% lang('footer_producers') %]</a></li>
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we have room for this link at all resolutions / all languages. Maybe we could display it only on very large screens.

image

Copy link
Member

Choose a reason for hiding this comment

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

@aadak99 do you think you can tackle this ?

It's about adding a class "show-for-xlarge-up" for example. See https://get.foundation/sites/docs-v5/components/visibility.html

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@stephanegigandet @alexgarel
I should be able to handle this as it is just an addition of a class. I'll have it done by tonight EST!

@github-actions github-actions bot added the 💥 Merge Conflicts 💥 Merge Conflicts label Jun 6, 2023
Copy link
Contributor

@stephanegigandet stephanegigandet left a comment

Choose a reason for hiding this comment

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

Thank you!

@@ -64,6 +64,7 @@

<li><label>[% lang("resources") %]</label></li>
<li>
<li><a href="https://www.youtube.com/playlist?list=PLjAH-USadsF3xgSVfUUBS4we3XBr-1r55" target="_blank">lang("video_tutorials")</a></li>
Copy link
Contributor

Choose a reason for hiding this comment

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

This part of the code was recently moved by @MonalikaPatnaik in another file: web/common/producers_resources_list.tt.html , so we should add this new list item there.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Done!

teolemon and others added 11 commits June 8, 2023 18:30
* Make user deletion a minion job

* Change message

* Change way enqueue is called to avoid issues with tests

* Perltidy fix

* Rever PO file changes
…openfoodfacts#8479)

Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 5.59.7 to 5.59.8.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v5.59.8/packages/parser)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/parser"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
We've never exported the recent_changes collection, which includes the
history of all changes performed on OFF DB since the
11th of September, 2018.

It's the most interesting data source to analyze contributor/apps
impacts.

This commit adds the automatic export of this collection as a JSONL
gzipped file.
@github-actions github-actions bot added 🧬 Taxonomies https://wiki.openfoodfacts.org/Global_taxonomies 🧪 tests Text 🧪 unit tests 👥 Users labels Jun 8, 2023
@github-actions github-actions bot removed Products GitHub Actions Pull requests that update Github_actions code Text 🧽 Data quality https://wiki.openfoodfacts.org/Quality 🧬 Taxonomies https://wiki.openfoodfacts.org/Global_taxonomies 🧪 tests Tags 🥗 Ingredients 👥 Users Minion categories 🧪 unit tests dependencies Pull requests that update a dependency file NGINX multilingual products 🏭 Producers Platform https://wiki.openfoodfacts.org/Platform_for_producers labels Jun 8, 2023
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 8, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug B 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot E 1 Security Hotspot
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

@stephanegigandet stephanegigandet left a comment

Choose a reason for hiding this comment

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

Thank you very much @aadak99 and @MonalikaPatnaik !

@stephanegigandet stephanegigandet merged commit da644c3 into openfoodfacts:main Jun 8, 2023
@aadak99
Copy link
Contributor Author

aadak99 commented Jun 8, 2023

Thank you very much @aadak99 and @MonalikaPatnaik !

@alexgarel @MonalikaPatnaik @stephanegigandet
I appreciate everyone's help and patience with me, now that I understand mostly the process I will be helping out where I can!

@aadak99 aadak99 deleted the htmlYouTubeTutorialsLink branch June 8, 2023 18:53
@stephanegigandet
Copy link
Contributor

I appreciate everyone's help and patience with me, now that I understand mostly the process I will be helping out where I can!

Thank you so much @aadak99! :)

@alexgarel
Copy link
Member

Thanks for your contribution @aadak99, really appreciated !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💥 Merge Conflicts 💥 Merge Conflicts product history We have kept 10 years of product revisions. This is useful to monitor edits & product improvements Template::Toolkit The templating toolkit used by product opener. The starting point for HTML/JS/CSS fixes. Translations We use a non-standard version of GetText, lack language variants support translate.openfoodfacts.org
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a link to the Tutorials in the menu of the Pro platform