Mudlet icon indicating copy to clipboard operation
Mudlet copied to clipboard

Automate translator attribution in Mudlet releases

Open vadi2 opened this issue 3 years ago • 5 comments
trafficstars

Brief summary of issue / Description of requested feature:

We've got a line saying Thanks to all translators ... in every news post to be grateful to everyone who's kept translations up to date in this update - which is composed from the names visible on https://crowdin.com/project/mudlet/activity-stream. It should be possible to automate this to get all names automatically from Crowdin (API or webpage), sort them alphabetically, and concatenate it with commas.

vadi2 avatar Jan 23 '22 05:01 vadi2

Hi, @vadi2 could you try this curl command using an admin token. I personally get a 403 forbidden response but I think you should be able to get the results as the project owner in crowdin. If that's right would it be possible to get such a token for my tests ? Otherwise I would need to see what the results look like but I think it will require to retrieve data from other forbidden endpoints.

curl -H "Authorization: Bearer <your_admin_token>" https://api.crowdin.com/api/v2/projects/306263/members

FloXire avatar Feb 19 '22 22:02 FloXire

I've created it, but we need a secure way to share it. I'm vadimp on keybase, can you get on that and I'll send it over to you?

vadi2 avatar Feb 20 '22 06:02 vadi2

Alright I'm floxire. I sent you a message.

FloXire avatar Feb 20 '22 14:02 FloXire

Sent.

vadi2 avatar Feb 20 '22 14:02 vadi2

The https://support.crowdin.com/api/v2/#operation/api.projects.reports.post 'top members' report can be used to automate this!

vadi2 avatar Apr 27 '22 08:04 vadi2

What is the status here?

Kebap avatar Nov 23 '22 16:11 Kebap

What is the status here?

Hi I'm not working on it and I don't think I'll be able to anytime soon unfortunately.

FloXire avatar Nov 23 '22 19:11 FloXire

Alright thanks for the update FloXire!

Kebap avatar Nov 23 '22 19:11 Kebap

/bounty $30

vadi2 avatar May 29 '23 19:05 vadi2

~~💎 $120 bounty created by Mudlet~~ ~~🙋 If you start working on this, comment /attempt #5919 to notify everyone~~ ~~👉 To claim this bounty, submit a pull request that includes the text /claim #5919 somewhere in its body~~ ~~📝 Before proceeding, please make sure you can receive payouts in your country~~ ~~💵 Payment arrives in your account 2-5 days after the bounty is rewarded~~ ~~💯 You keep 100% of the bounty award~~ ~~🙏 Thank you for contributing to Mudlet/Mudlet!~~

Attempt Started (GMT+0) Solution
🔴 @tachyonicbytes May 29, 2023, 7:43:33 PM WIP
🔴 @rajdip-b Dec 10, 2023, 12:52:17 PM WIP
🟢 @wiploo Dec 12, 2023, 10:44:10 AM #1

algora-pbc[bot] avatar May 29 '23 19:05 algora-pbc[bot]

/attempt #5919

Can I work on two in parallel?

tachyonicbytes avatar May 29 '23 19:05 tachyonicbytes

I don't see why not, since anyone else is still welcome to come submit their solutions while you do so.

Thanks for taking this up!

vadi2 avatar May 29 '23 20:05 vadi2

I may also need that key that you referred to before. My keybase id is https://keybase.io/tachyonicbytes

tachyonicbytes avatar May 31 '23 20:05 tachyonicbytes

/bounty $120

vadi2 avatar Dec 10 '23 08:12 vadi2

/attempt #5919

Options

rajdip-b avatar Dec 10 '23 12:12 rajdip-b

Hi @vadi2, I have some question about this bounty, maybe are quite obvius, but I can't find any information about:

  • an example of "thanks for all translators..." is the final string in the news https://www.mudlet.org/2019/11/4-3-fixed-selection/ ?
  • the translators string can be generated in wordpress using a shortcode or calling a special webpage or using a command line script or... ?
  • can you solve the problema about the /attempt because i think that @tachyonicbytes is not working about it anymore.

Thanks

Options

wiploo avatar Dec 11 '23 13:12 wiploo

Hi @wiploo

an example of "thanks for all translators..." is the final string in the news https://www.mudlet.org/2019/11/4-3-fixed-selection/ ?

In your example, this would be that line: Thanks to all translators: Leris, Marco “M0lid3us” Tironi (wiploo), and vadi2 for helping translate Mudlet.

Kebap avatar Dec 11 '23 13:12 Kebap

I can confirm that I am not working on this particular issue anymore. If there is a cancel button or anything that I have to click so that @wiploo can be certified he is the sole assigned person, please let me know

tachyonicbytes avatar Dec 11 '23 13:12 tachyonicbytes

Note

The user @tachyonicbytes is already attempting to complete issue #5919 and claim the bounty. We recommend checking in on @tachyonicbytes's progress, and potentially collaborating, before starting a new solution.

Damn... I hade a mistake using bot command. I don't want to attempt because @rajdip-b is already working on it and asked before me. Sorry.

wiploo avatar Dec 11 '23 13:12 wiploo

Note The user @tachyonicbytes is already attempting to complete issue #5919 and claim the bounty. We recommend checking in on @tachyonicbytes's progress, and potentially collaborating, before starting a new solution.

Damn... I hade a mistake using bot command. I don't want to attempt because @rajdip-b is already working on it and asked before me. Sorry.

hey @wiploo, i started to attempt it, but will no longer be able to finish it. you can take it up! strange thing that the attempt wasn't cancelled!

rajdip-b avatar Dec 11 '23 13:12 rajdip-b

hey folks, sorry for the outdated bot warnings! we've cleaned up all inactive attempts

algora-pbc avatar Dec 11 '23 15:12 algora-pbc

@vadi2 can you reply to this https://github.com/Mudlet/Mudlet/issues/5919#issuecomment-1850107675 (point 2-3)? Thanks!

wiploo avatar Dec 11 '23 21:12 wiploo

The translators string can be generated by calling the crowdin API and getting the names of translators since the last release: https://crowdin.com/project/mudlet/activity-stream

Try claiming again? Not sure why the first one didn't work.

vadi2 avatar Dec 12 '23 05:12 vadi2

/attempt https://github.com/Mudlet/Mudlet/issues/5919

Options

wiploo avatar Dec 12 '23 10:12 wiploo

The translators string can be generated by calling the crowdin API and getting the names of translators since the last release: https://crowdin.com/project/mudlet/activity-stream

Try claiming again? Not sure why the first one didn't work.

Hi @vadi2, can you please tell me the last release and/or how it is marked in the activity-stream page? (eventually we can set manually the date in the script)

Do you like to have ALL translator listed in that time frame or only the best X? (they are however auto-arranged from most to least contributor).

Best regards

wiploo avatar Dec 13 '23 22:12 wiploo

Hey! Crowdin does not know about releases, but Github does - both via /releases and by Git tagging.

All translators in the timeframe should be listed, sorted alphabetically. This means that those with non-Latin names end up at the end usually.

vadi2 avatar Dec 14 '23 07:12 vadi2

Hey! Crowdin does not know about releases, but Github does - both via /releases and by Git tagging.

All translators in the timeframe should be listed, sorted alphabetically. This means that those with non-Latin names end up at the end usually.

Ok, I'll look to github api to get the lastest release, but 4.17.2 is not listed !?!? Maybe a manual-inserted date is more effective?

wiploo avatar Dec 14 '23 07:12 wiploo

Ok, look at tags instead: https://github.com/Mudlet/Mudlet/tags, those are always there. The missing release for the tag is a mistake - it's a manual process...

vadi2 avatar Dec 14 '23 07:12 vadi2

Unfortunatelly, github api don't provide the date of a tag. Shall we use the release informations even if incomplete or do you prefear to manually insert the date?

https://docs.github.com/en/rest/repos/repos?apiVersion=2022-11-28#list-repository-tags https://docs.github.com/en/rest/releases/releases?apiVersion=2022-11-28#get-the-latest-release

wiploo avatar Dec 14 '23 09:12 wiploo

Let's go with a release then, because those should be there - as well as an option to manually insert the date.

vadi2 avatar Dec 14 '23 10:12 vadi2