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

Show the strings of a translation event #198

Merged
merged 32 commits into from
Apr 30, 2024
Merged

Show the strings of a translation event #198

merged 32 commits into from
Apr 30, 2024

Conversation

amieiro
Copy link
Collaborator

@amieiro amieiro commented Apr 9, 2024

Fixes #169. Needs GlotPress/gp-translation-helpers#222.

This adds links to the Translations table:
Screenshot 2024-04-17 at 16 43 05

For example for the WordCamp Asia contributor day, this now adds new URLs like:

and that page then lists all the strings that were modified in the project:
Screenshot 2024-04-17 at 17-37-29 Translation Events – WordCamp Asia 2024 Contributor Day WordPress org

Here you can see how this also takes into account individual project permissions. I am a PTE for German in the Friends plugin, but not a GTE for German, thus cannot approve in Core:

Screenshot 2024-04-17 at 17-36-34 akirk dev ord Translation Events – WordCamp Asia 2024 Contributor Day WordPress org

includes/stats-calculator.php Outdated Show resolved Hide resolved
templates/event-translations.php Outdated Show resolved Hide resolved
@akirk akirk marked this pull request as ready for review April 17, 2024 15:39
@akirk akirk requested a review from psrpinto April 17, 2024 16:03
Copy link
Member

@psrpinto psrpinto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm getting a 404 when trying to access the translations page, by clicking in an entry in the stats table. The URL is:

http://localhost:8888/events/test-hosts/pt/

Not sure why it's 404ing.

wporg-gp-translation-events.php Outdated Show resolved Hide resolved
includes/routes/event/translations.php Outdated Show resolved Hide resolved
includes/routes/event/translations.php Outdated Show resolved Hide resolved
includes/routes/event/translations.php Outdated Show resolved Hide resolved
includes/routes/event/translations.php Outdated Show resolved Hide resolved
@akirk
Copy link
Member

akirk commented Apr 17, 2024

@psrpinto could you try again with the latest regex updates?

@psrpinto
Copy link
Member

psrpinto commented Apr 18, 2024

The 404 is still there but I now realised why it's happening: it lacks the /glotpress prefix, it's /events/$event-slug/translations/pt/ but it should be /glotpress/events/$event-slug/translations/pt/.

templates/event.php Outdated Show resolved Hide resolved
@akirk akirk requested a review from psrpinto April 24, 2024 06:15
Copy link
Member

@psrpinto psrpinto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and seems to work well. Great feature!

@akirk akirk merged commit 5910bf5 into trunk Apr 30, 2024
3 checks passed
@akirk akirk deleted the event-strings branch April 30, 2024 09:24
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.

See the unreviewed strings of a translation event
3 participants