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
8244824: TableView : Incorrect German translation #220
Conversation
👋 Welcome back aghaisas! A progress list of the required criteria for merging this PR into |
FYI: While I agree "Inhalt" is better than "Content" the latter is not completely wrong (see https://www.duden.de/rechtschreibung/Content) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct :)
@kevinrushforth brought to my notice that there is a PR (#210) opened for the same issue by schelldorfer. I was not aware of this PR as it did not have 'rfr' label - which I look for. I would NOT like to step on a new contributor's contribution. Now, I see that the original PR has been closed. Once his signed OCA has been recorded, we have two options :
|
@aghaisas that sounds fine to me. In order to attribute the fix, you will need to wait for the OCA to be recorded. @schelldorfer once your OCA is recorded, which would you prefer to do? Create a new PR yourself or have @aghaisas list you as the contributor. |
I signed the OCA a few days ago and my name is listed on the page |
@aghaisas can you proceed with this one or is there anything required from my side? |
I need confirmation from you about which option you prefer (See my last comment above). @kevinrushforth also has asked the same question in his last comment. |
let's do option 1 |
@aghaisas This change now passes all automated pre-integration checks. When the change also fulfills all project specific requirements, type
Since the source branch of this PR was last updated there have been 24 commits pushed to the
As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid automatic rebasing, please merge ➡️ To integrate this PR with the above commit message to the |
/contributor schelldorfer |
@aghaisas Syntax:
|
/contributor @schelldorfer |
@aghaisas Syntax:
|
/contributor add @schelldorfer |
@aghaisas Could not parse
|
/contributor add Martin Schelldorfer schelldorfer@gmail.com |
@aghaisas |
/integrate |
@aghaisas The following commits have been pushed to master since your change was applied:
Your commit was automatically rebased without conflicts. Pushed as commit b200891. |
Issue : https://bugs.openjdk.java.net/browse/JDK-8244824
Fix : As simple as it gets !!!
Progress
Issue
Reviewers
Contributors
<schelldorfer@gmail.com>
Download
$ git fetch https://git.openjdk.java.net/jfx pull/220/head:pull/220
$ git checkout pull/220