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.

Sign up for GitHub

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

Jump to bottom

fix: Add contributors link in user profile for easier account deletion #8399

Merged
merged 3 commits into from
May 9, 2023
Merged

fix: Add contributors link in user profile for easier account deletion #8399

merged 3 commits into from
May 9, 2023

Conversation

teolemon
Copy link
Member

@teolemon teolemon commented May 5, 2023 •
edited
Loading

What

fix: Add contributors link in user profile for easier account deletion

Part of

@teolemon teolemon changed the title Update Display.pm fix: Add contributors link in user profile for easier account deletion May 5, 2023
@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 5, 2023
@teolemon teolemon marked this pull request as ready for review May 5, 2023 17:12
@teolemon teolemon requested a review from a team as a code owner May 5, 2023 17:12
@teolemon teolemon requested a review from alexgarel May 5, 2023 17:12
@sonarcloud
Copy link

sonarcloud bot commented May 5, 2023

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
No Duplication information No Duplication information

@codecov-commenter
Copy link

codecov-commenter commented May 5, 2023 •
edited
Loading

Codecov Report

Merging #8399 (4950091) into main (e70a705) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main    #8399   +/-   ##
=======================================
  Coverage   48.44%   48.44%           
=======================================
  Files         113      113           
  Lines       21251    21251           
  Branches     4767     4767           
=======================================
  Hits        10296    10296           
  Misses       9674     9674           
  Partials     1281     1281
Impacted Files Coverage Δ
lib/ProductOpener/Display.pm 9.88% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

alexgarel
alexgarel approved these changes May 9, 2023
View reviewed changes
Copy link
Member

@alexgarel alexgarel left a comment

Choose a reason for hiding this comment

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

👍

@alexgarel alexgarel merged commit 232b32c into main May 9, 2023
@alexgarel alexgarel deleted the contributors-link-in-user-profile branch May 9, 2023 14:24
@openfoodfacts-bot openfoodfacts-bot mentioned this pull request May 9, 2023
Merged
@teolemon teolemon added the Account deletion Enabling users to delete account while preserving accountability for edits label May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexgarel alexgarel alexgarel approved these changes

Assignees

@teolemon teolemon

Labels
Account deletion Enabling users to delete account while preserving accountability for edits Display Translations We use a non-standard version of GetText, lack language variants support translate.openfoodfacts.org
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@teolemon @codecov-commenter @alexgarel