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

fix: added translation changes #1939

Merged
merged 1 commit into from
Jan 2, 2024
Merged

fix: added translation changes #1939

merged 1 commit into from
Jan 2, 2024

Conversation

PC-11-00
Copy link
Collaborator

@PC-11-00 PC-11-00 commented Dec 15, 2023

Description

Now we can see the Client Approval, actually translation spelling was wrong so Html file was not loading and added few more translation changes

Related issues and discussion

Screenshots, if any

i35.mov

Checklist

Please make sure these boxes are checked before submitting your pull request - thanks!

  • If you have multiple commits please combine them into one commit by squashing them.

  • Read and understood the contribution guidelines at web-app/.github/CONTRIBUTING.md.

@bharathcgowda
Copy link
Collaborator

@adamsaghy could you please help us in reviwing this PR?

@adamsaghy
Copy link
Collaborator

@PC-11-00 @bharathcgowda I see no other changes but code format changes and extra translations. Is there anything i am missing?

@PC-11-00
Copy link
Collaborator Author

PC-11-00 commented Jan 2, 2024

No @adamsaghy, only translations changes

@adamsaghy
Copy link
Collaborator

adamsaghy commented Jan 2, 2024

No @adamsaghy, only translations changes

Would you be so kind to rename the PR title, PR description and commit message accordingly, please? It is pretty misleading now...

@PC-11-00 PC-11-00 changed the title fix: now checker and maker is working for clientApproval fix: added translation changes Jan 2, 2024
@PC-11-00
Copy link
Collaborator Author

PC-11-00 commented Jan 2, 2024

@adamsaghy yeah okay, actually I was working on this(maker-checker) issue, and by mistake I forgot to change the name sorry

@adamsaghy adamsaghy merged commit e87598a into openMF:master Jan 2, 2024
1 of 2 checks passed
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