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

Jump to bottom

fix: translation guide updates #157

Merged
merged 1 commit into from Oct 26, 2023
Merged

fix: translation guide updates #157

merged 1 commit into from Oct 26, 2023

Conversation

jdwilkin4
Copy link
Contributor

@jdwilkin4 jdwilkin4 commented Oct 26, 2023 •
edited

Description

This PR accomplishes the following:

  • renames the translation guidelines file to the correct name of i18n instead of i18tn
  • adds the link for the guidelines file in the readme

What type of PR is this? (check all applicable)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🎨 Style
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build
  • 🔁 CI
  • 📦 Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

closes #144

Mobile & Desktop Screenshots/Recordings

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

Added to documentation?

  • 📜 README.md
  • 📓 docs.opensauced.pizza
  • 🍕 dev.to/opensauced
  • 📕 storybook
  • 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

CBID2
CBID2 approved these changes Oct 26, 2023
View reviewed changes
Copy link
Member

@CBID2 CBID2 left a comment

Choose a reason for hiding this comment

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

Looks good! :)

@CBID2 CBID2 added documentation Improvements or additions to documentation 🐛 bug Something isn't working labels Oct 26, 2023
@CBID2 CBID2 merged commit de3c5d1 into open-sauced:main Oct 26, 2023
1 check passed
@CBID2 CBID2 mentioned this pull request Oct 26, 2023
Merged
19 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CBID2 CBID2 CBID2 approved these changes

Assignees

@jdwilkin4 jdwilkin4

Labels
🐛 bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Bug: Translation Guide updates
2 participants
@jdwilkin4 @CBID2