Skip to content

feat(API): Support omit_translation_keys param in Jobs#show#944

Merged
bikmazefe merged 2 commits intomainfrom
scd-407
Oct 27, 2025
Merged

feat(API): Support omit_translation_keys param in Jobs#show#944
bikmazefe merged 2 commits intomainfrom
scd-407

Conversation

@bikmazefe
Copy link
Member

@bikmazefe bikmazefe marked this pull request as ready for review October 23, 2025 11:49
@bikmazefe bikmazefe requested review from a team and jablan and removed request for jablan October 23, 2025 11:49
parameters.yaml Outdated
omit_translation_keys:
in: query
name: omit_translation_keys
description: Omit translation keys in the response
Copy link
Collaborator

Choose a reason for hiding this comment

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

perhaps should be explained why would someone want to use this :)

@bikmazefe
Copy link
Member Author

bikmazefe commented Nov 10, 2025

Hey @lookasc this change is not visible in the docs, as far as I can see the dispatch workflow was successful, any idea maybe what could be the issue? 🤔

https://github.com/phrase/openapi/actions/runs/18844013562/job/53763231769

@jablan
Copy link
Collaborator

jablan commented Nov 10, 2025

hey @bikmazefe where is this parameter used?

@jablan
Copy link
Collaborator

jablan commented Nov 10, 2025

@bikmazefe
Copy link
Member Author

shouldn't it have been added to

https://github.com/phrase/openapi/blob/95b6c2a2ab798d01f9bc0d53a22dbf817a5eb0ee/paths/jobs/show.yaml#L12

@jablan oh my 🤦 I'm definitely blind, I'll open a PR, thank you!

@jablan
Copy link
Collaborator

jablan commented Nov 10, 2025

@bikmazefe well it was reviewed by another two of us, so 🙈 🙈 🙈

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.

3 participants