Add eprinttype field to General Entry editor tab#14620
Conversation
|
Hey @prerak09! 👋 Thank you for contributing to JabRef! We have automated checks in place, based on which you will soon get feedback if any of them are failing. After all automated checks pass, a maintainer will also review your contribution. Once that happens, you can go through their comments in the "Files changed" tab and act on them, or reply to the conversation if you have further inputs. Please re-check our AI Usage Policy to ensure that your pull request is in line with it. It also contains links to our contribution guide in case of any other doubts related to our contribution workflow. |
koppor
left a comment
There was a problem hiding this comment.
Good first step. However, there already is a similar method. Please unify. Comment inline.
|
@koppor i forgot to remove the comments that i usually do to see what new things i added. |
|
I unified the migration by extending |
|
@koppor i changed the method name from icore to migrateGeneralTabDefaultFields now the method migrates the General tab defaults when the user still has an old default setup.
|
java docs change to what the method is currently doing
…ns.java Co-authored-by: Subhramit Basu <subhramit.bb@live.in>
Closes #14555
This pull request adds the
eprinttypefield to the default General entry editor tab so that it appears alongsideeprint, which is where users naturally expect it.In addition, a preference migration is included to ensure that existing users who still use the old default General tab configuration are updated automatically, while fully respecting any custom tab setups.
Steps to test
eprintandeprinttypefields are visible in the General tab.eprintis added or not .Mandatory checks
CHANGELOG.mdin a way that is understandable for the average user