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

[FR] Fixes before merge #2288

Merged
merged 6 commits into from
Sep 6, 2023
Merged

[FR] Fixes before merge #2288

merged 6 commits into from
Sep 6, 2023

Conversation

sestegra
Copy link
Collaborator

@sestegra sestegra commented Aug 11, 2023

Describe your changes

  • Fix frontmatter that aren't aligned with main branch
  • Fix internal links to point on French version
  • Fix dead links
  • Add missing links
  • Fix general errors
  • Localize tags

Related issue number or link (ex: resolves #issue-number)

N/A

Checklist before opening this PR (put x in the checkboxes)

  • This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Signed-off-by: Stéphane Este-Gracias <[email protected]>
@netlify
Copy link

netlify bot commented Aug 11, 2023

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit 0c57b10
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/64da7d2ac907ec0008f6d598
😎 Deploy Preview https://deploy-preview-2288--cncfglossary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added the lang/fr for French label Aug 11, 2023
@sestegra sestegra self-assigned this Aug 11, 2023
@sestegra sestegra added this to the [French] v1 milestone Aug 11, 2023
@sestegra sestegra changed the title [FR] Fix frontmatter [FR] Fix frontmatters before merge Aug 11, 2023
@sestegra sestegra marked this pull request as draft August 11, 2023 17:11
@sestegra sestegra changed the title [FR] Fix frontmatters before merge [FR] Fixes before merge Aug 11, 2023
Signed-off-by: Stéphane Este-Gracias <[email protected]>
Signed-off-by: Stéphane Este-Gracias <[email protected]>
Signed-off-by: Stéphane Este-Gracias <[email protected]>
@sestegra sestegra marked this pull request as ready for review August 11, 2023 19:03
Copy link
Collaborator

@jihoon-seo jihoon-seo left a comment

Choose a reason for hiding this comment

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

@sestegra Thank you for your hard work!

@@ -5,7 +5,7 @@ category: Property
tags: ["fundamental", "", ""]
Copy link
Collaborator

Choose a reason for hiding this comment

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

How about localizing tags? 😊
e.g., https://glossary.cncf.io/pt-br/tags/

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Indeed, I put the PR back to Draft to localize the tags.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I just pushed a new commit to localize the tags

@sestegra sestegra marked this pull request as draft August 12, 2023 00:00
Signed-off-by: Stéphane Este-Gracias <[email protected]>
@sestegra sestegra marked this pull request as ready for review August 14, 2023 19:16
Copy link
Collaborator

@huats huats left a comment

Choose a reason for hiding this comment

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

Thanks a lot for such work ! it looks great !

@sestegra sestegra merged commit 0397969 into cncf:dev-fr Sep 6, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/fr for French
Projects
Development

Successfully merging this pull request may close these issues.

4 participants