westberliner / checksum

Plugin for Nextcloud and ownCloud to create hashes of files.
GNU General Public License v3.0
29 stars 15 forks source link

Clarify process to join Transifex via URL in docs #70

Closed BjoernLudwigPTB closed 1 year ago

BjoernLudwigPTB commented 1 year ago

Thanks for the great work here!

I noticed you added the project to transifex and everything seemes to have worked on nextcloud side as well as of 5 Jan 2021. Anyway now the transifex project URL you share in the README does not work anymore and I was not able to find your project there: https://explore.transifex.com/search/?q=checksum.

p-bo commented 1 year ago

@BjoernLudwigPTB resource for this app is normally there obrazek and URL you mentioned is functional.

BjoernLudwigPTB commented 1 year ago

Ah okay, that just works after visiting https://explore.transifex.com/nextcloud/nextcloud/ and there clicking "Join this project". Otherwise you get as only answer:

{"message": "Forbidden"}

when visiting https://www.transifex.com/nextcloud/search/?q=resource%3Achecksum and searching for checksum anywhere on transifex.com be it in- or outside of the nextcloud organisation yields no results.

Thanks for just confirming that everything should work. I would recommend to mention the necessity to join the nextcloud organization before being able to visit your link and will provide a short PR, if you like.

p-bo commented 1 year ago

I see, so correct URL, where to translate is https://www.transifex.com/nextcloud/nextcloud/checksum/ then one will get either Transifex login page, or Join Nextcloud project (cannot walk trough, because I`m already joined by my account). In both cases when one is joined into Nextcloud project on Transifex, there is then Translate button and one can start translating/improving. So I you want, please do PR with this URL (perhaps that it is needed to join project there on Transifex - but it is standard on this platform).

BjoernLudwigPTB commented 1 year ago

I changed the title. Could you please assign me?

p-bo commented 1 year ago

@BjoernLudwigPTB only maintainers of this repository can do this (not my case - mere translator). But you can do PR with fix of readme regardless of assignment of this issue report. Maintainers then will review and (if it fits) merge it.

BjoernLudwigPTB commented 1 year ago

@BjoernLudwigPTB only maintainers of this repository can do this (not my case - mere translator). But you can do PR with fix of readme regardless of assignment of this issue report. Maintainers then will review and (if it fits) merge it.

Ah yes, sure… I did not pay any attention with whom I am communicating here. Sorry, to bother you. Of course this is not a blocker for contributing, just would help me to organise things.

@westberliner Could you assign me?

westberliner commented 1 year ago

Hi @BjoernLudwigPTB,

if I understand you correctly, do you want to update the Readme? Feel free to create a PR.

Alternatively how about this:

Translation
-----------

If your language is missing, feel free to translate your language under [transifex](https://www.transifex.com/nextcloud/nextcloud/checksum/).

If you are not already a member of the transifex nextcloud project create an account and join the project.

Does this sound correct to you? Thx for hinting this out.

BjoernLudwigPTB commented 1 year ago

@westberliner Your suggestion sounds good to me. At least it is the until now missing hint, that there is a preceeding step, to being able to collaborate.

BjoernLudwigPTB commented 1 year ago

if I understand you correctly, do you want to update the Readme?

That's right, still. Assign me, and I will go for it, unless you have already applied your suggested change.

westberliner commented 1 year ago

Readme updated. Thx for mentioning this.