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

RUN-2136: Fix step and node step plugin translation #8802

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

leandromqrs
Copy link
Contributor

Is this a bugfix, or an enhancement? Please describe.
Bug fix when try to translate the step and node step plugins to another language.

Describe the solution you've implemented
When the messagePrefix is null, it puts the value of the property as nullproperty.something.title. After the fix, with the verification if is null, it puts with the correct value property.something.title

Describe alternatives you've considered

Additional context
Additional images to compare the before and after...

  • Before
    Screenshot from 2024-01-08 22-31-28

  • After
    Screenshot from 2024-01-08 22-41-52

  • Before
    Screenshot from 2024-01-08 22-42-31

  • After
    Screenshot from 2024-01-08 22-42-53

@fdevans fdevans added the to-jira label Jan 9, 2024
@fdevans fdevans requested a review from a team January 9, 2024 03:27
@fdevans fdevans added this to the 5.1.0 milestone Jan 9, 2024
@fdevans fdevans changed the title Fix step and node step plugin translation RUN-2136: Fix step and node step plugin translation Jan 9, 2024
@ltamaster ltamaster modified the milestones: 5.1.0, 5.2.0 Jan 18, 2024
@leandromqrs leandromqrs force-pushed the fix-plugin-translation branch 2 times, most recently from d9d841f to 9bc06ec Compare January 18, 2024 23:37
@leandromqrs leandromqrs force-pushed the fix-plugin-translation branch 4 times, most recently from 40e4ef4 to 7d88005 Compare February 2, 2024 21:23
@charles014
Copy link

I can't wait for it to be merged so I can contribute with my translations.

@leandromqrs leandromqrs force-pushed the fix-plugin-translation branch 4 times, most recently from a2cb6b9 to f7563d1 Compare February 9, 2024 20:52
@leandromqrs leandromqrs force-pushed the fix-plugin-translation branch 3 times, most recently from 3218223 to 28841d2 Compare February 20, 2024 19:57
@leandromqrs leandromqrs force-pushed the fix-plugin-translation branch 3 times, most recently from cf9201e to 630a51d Compare February 27, 2024 12:31
@leandromqrs leandromqrs force-pushed the fix-plugin-translation branch 2 times, most recently from c40784a to 80e98c9 Compare March 7, 2024 18:55
@ltamaster ltamaster removed this from the 5.2.0 milestone Mar 26, 2024
@leandromqrs
Copy link
Contributor Author

@ltamaster sorry for the question, but, why it is not merged yet ? It's a very simple fix 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants