-
Notifications
You must be signed in to change notification settings - Fork 15
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
feature request: add "no ranking" subgroup automatically #10264 #699
Conversation
Please submit to https://github.com/JabRef/jabref/pulls |
I believe I still need other members to review my code. I'm a bit confused - I created the pull request in my fork repository. Could you kindly explain why you received this pull request? I understand I should submit the PR to the JabRef main repository at https://github.com/JabRef/jabref/pulls instead, but I want to make sure I'm following the correct process. |
Yeah, nice!
You, did the mistake to work on the Quick workaround: I repon the PR and your colleagues can comment here. After they finished, You can submit to JabRef. Longer workaround: You rename your branch:
After these steps, you can create a pull request to your own repository. |
I enabled the "quick woraround". If you intend to follow the "Longer workaround", please let me know. |
Lych... If you reference the issue with a full link in your commit, GitHub recognizes this. I cannot do anythign against it. YOU created a pull request. One can create a pull request even if the issue is not assigned. GitHub is an open eco system allowing "everyone" to contribute. Even without asking before. |
What? Which pull request? This one? It is still open. If you mean another, please add the link here.
No. This is NOT GitHub working style.
You can continue. Please, before you ask your teammates to review, please try to fix on your own:
|
a94dd11
to
8cf8959
Compare
Add "no ranking" subgroup automatically when creating ranking groups
When creating a ranking group based on the "ranking" keyword, there is currently no subgroup created for entries without ranking. Since it's useful to easily find the entries without rank, this PR adds a "no ranking" subgroup that is created automatically alongside rank1-rank5 subgroups.
Closes JabRef#10264
Changes in this pull request
Implementation Details
GroupDialogViewModel.resultConverter()
to detect when creating ranking-based groupsTest Scenario
Mandatory checks
CHANGELOG.md
described in a way that is understandable for the average userScreenshots
[Add before/after screenshots showing the new "no ranking" subgroup]