Skip to content

Translation/fa #54

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

Closed
wants to merge 52 commits into from
Closed

Conversation

This-is-Neo
Copy link
Contributor

@PauloASilva Hello Dear Paulo! Persian Translation is done.

@PauloASilva
Copy link
Collaborator

@This-is-Neo I've done a quick check on changed files: the source files should be kept in Markdown format but all your files have a .docx extension. Can you please fix it?

@This-is-Neo
Copy link
Contributor Author

@PauloASilva Yes of course.

@PauloASilva
Copy link
Collaborator

@This-is-Neo following the OWASP vendor-neutral approach, I'll have to kindly ask you to remove any reference to RNP such as the one found in the Acknowledgements sections. We will be glad to credit those responsible for the translation, linking their name/nickname to a social profile (e.g. Twitter, GitHub, Linkedin) as you can see here.

Best,
Paulo A. Silva

@This-is-Neo
Copy link
Contributor Author

@PauloASilva Hello Dear Paulo!
MD Files were added and RNPG logo was removed.
I would be glad to credit the following:
Alireza Mostame: https://www.linkedin.com/in/alireza-mostame-29970b242
Mohammad Reza Ismaeli Taba: https://www.linkedin.com/in/rezataba
Amirmahdi Nowbakht: https://www.linkedin.com/in/amirmahdi-nowbakht-3b8865200
RNPG: https://www.linkedin.com/company/raspina-net-pars/

@This-is-Neo
Copy link
Contributor Author

@PauloASilva
@ErezYalon
Dear Paulo and Erez! I would be glad if you could proceed with approving the pull request or inform us about possible changes that need to be applied on the work.
Thanks :)

@PauloASilva
Copy link
Collaborator

PauloASilva commented Jul 29, 2022

Hi @This-is-Neo,
Although there still a reference to RNPG in the ODT and PDF we've decided to move forward merging the translation.
In the future we should publish some translation guidelines to avoid such issues and manage contributors expectations.

Cheers,
Paulo A. Silva

@This-is-Neo
Copy link
Contributor Author

Dear @PauloASilva We are happy to be a small part of this project and we're ready to be more engaged in upcoming versions and tasks, wherever and however you see fit.
Thanks :)

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