Skip to content

Conversation

@neelchauhan
Copy link
Contributor

@neelchauhan neelchauhan commented Jan 15, 2024

Author checklist (Completed by original Author)

  • Good fit for the Rocky Linux project? Title and Author Metatags inserted ?
  • If applicable, steps and instructions have been tested to work
  • Initial self-review to fix basic typos and grammar completed

Rocky Documentation checklist (Completed by Rocky team)

  • 1st Pass (Document is good fit for project and author checklist completed)
  • 2nd Pass (Technical Review - check for technical correctness)
  • 3rd Pass (Detailed Editorial Review and Peer Review)
  • Final approval (Final Review)

@sspencerwire
Copy link
Contributor

sspencerwire commented Jan 15, 2024

@neelchauhan Unfortunately (I think I alluded to this earlier) the translated files are not edited in this way. The source (English) files need to be changed and then our translation engine (Crowdin) will pick up the changes that our translators will massage and approve. If you want to make this change, modify the English (source) file instead: cloud_server_using_nextcloud.md and delete the other changes the .ko, .it, and .uk files from your PR.

If you don't want to change this PR, close it and just create a new one with the changes to the source file.

@neelchauhan
Copy link
Contributor Author

I've made the changes, please re-review/approve. Thanks for letting me know.

@github-actions
Copy link

Test results for b3ccb35:

Number of broken URLs: 2

URL,RESULT,FILENAME
 https://git.sr.ht/%7Equartsize/,failed,guides/contribute/README.md
 https://www.linode.com/docs/guides/beginners-guide-to-lxd-reverse-proxy,failed,guides/containers/lxd_web_servers.md

@sspencerwire
Copy link
Contributor

Looks good now!

@sspencerwire sspencerwire merged commit 66223e9 into rocky-linux:main Jan 15, 2024
@neelchauhan neelchauhan deleted the nextcloud-php81 branch November 29, 2024 18:52
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