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

Update best-practices.md #718

Closed
wants to merge 1 commit into from
Closed

Conversation

angelitaooo
Copy link

@angelitaooo angelitaooo commented Oct 6, 2018

Updated this file with missing spanish accents.
Thank you!

  • Have you followed the contributing guidelines?
  • Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.


Updated this file with missing spanish accents. 
Thank you!
@mlinksva
Copy link
Contributor

mlinksva commented Oct 6, 2018

Hi @angelitaooo I think the accents are there, just escaped. See https://opensource.guide/es/best-practices/ ... I don't know whether they need to be escaped or it would be more convenient to have unescaped text in the repo. Any thoughts on that (anyone)? The French translation seems to mostly (but not entirely) unescaped and seems to work. We should probably go one way or another and document in https://github.com/github/opensource.guide/blob/master/docs/translations.md

Also, the front matter keys between ---s (like title:) should not be translated...I guess https://github.com/github/opensource.guide/blob/master/docs/translations.md should say that too.

@mlinksva mlinksva closed this Oct 8, 2018
mlinksva added a commit that referenced this pull request Oct 8, 2018
Accounting for changed location of translation index files #716 in and clarify field names should not be translated #718 (comment)
@mlinksva mlinksva mentioned this pull request Oct 8, 2018
2 tasks
y777777 added a commit to y777777/opensource.guide that referenced this pull request Oct 18, 2018
* Fix form input for users with dark OS theme

* Add/update translation for tamil language

* Update for tamil translation

* Fix the lint issues

* Fix formatting (remove spaces before comma)

* Fix failing tests for links (tamil translation)

* Fix links not working (tamil translation)

* Fix failing links and typos

* Bump html-proofer from 3.9.1 to 3.9.2

Bumps [html-proofer](https://github.com/gjtorikian/html-proofer) from 3.9.1 to 3.9.2.
- [Release notes](https://github.com/gjtorikian/html-proofer/releases)
- [Commits](gjtorikian/html-proofer@v3.9.1...v3.9.2)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Prevent "time out" in script/html-proofer

External link ... failed: got a time out (response code 301)

* Bump primer-marketing from 6.2.4 to 6.2.5

Bumps [primer-marketing](https://github.com/primer/primer) from 6.2.4 to 6.2.5.
- [Release notes](https://github.com/primer/primer/releases)
- [Changelog](https://github.com/primer/primer/blob/master/CHANGELOG.md)
- [Commits](https://github.com/primer/primer/commits)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Bump github-pages from 188 to 189

Bumps [github-pages](https://github.com/github/pages-gem) from 188 to 189.
- [Release notes](https://github.com/github/pages-gem/releases)
- [Commits](github/pages-gem@v188...v189)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Add missing ToC entry

* avoid redirect

* add missing section link to translations

* section links to match translations

* missed one

* ta/legal: fix broken link.

* Bump github-pages from 189 to 191

Bumps [github-pages](https://github.com/github/pages-gem) from 189 to 191.
- [Release notes](https://github.com/github/pages-gem/releases)
- [Commits](github/pages-gem@v189...v191)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Update contributor funnel links.

Rather than link to the slides from a talk I did let's link to the blog
post explaining the concept a bit better.

Also, let's update the image to a sharper and more emojid one.

* [Security] Bump rubyzip from 1.2.1 to 1.2.2

Bumps [rubyzip](https://github.com/rubyzip/rubyzip) from 1.2.1 to 1.2.2. **This update includes security fixes.**
- [Release notes](https://github.com/rubyzip/rubyzip/releases)
- [Changelog](https://github.com/rubyzip/rubyzip/blob/master/Changelog.md)
- [Commits](rubyzip/rubyzip@v1.2.1...v1.2.2)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Update hyperlinks

* Update hyperlinks (fixup f0437ee)

* Bump github-pages from 191 to 192

Bumps [github-pages](https://github.com/github/pages-gem) from 191 to 192.
- [Release notes](https://github.com/github/pages-gem/releases)
- [Commits](github/pages-gem@v191...v192)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Update best-practices.md

Updated `procesimientos` since it's not a word :-), the right word could be `procesos`.

Also, I added a `í` that was missing 👍

* fix link

* Fix back to the index page link for non-English languages

* Add keep-all to headings and heading-like elements

* avoid redirect

* Move index files into their own language directories

* [Security] Bump nokogiri from 1.8.4 to 1.8.5

Bumps [nokogiri](https://github.com/sparklemotion/nokogiri) from 1.8.4 to 1.8.5. **This update includes security fixes.**
- [Release notes](https://github.com/sparklemotion/nokogiri/releases)
- [Changelog](https://github.com/sparklemotion/nokogiri/blob/master/CHANGELOG.md)
- [Commits](sparklemotion/nokogiri@v1.8.4...v1.8.5)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Fix Getting Paid page in Spanish language

* Update es.yml

I apply text that is still to be translated

* Update translation instructions

Accounting for changed location of translation index files github#716 in and clarify field names should not be translated github#718 (comment)
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.

2 participants