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

Adds a browse button next to the path text field for aux-based groups #4743

Merged
merged 6 commits into from
Mar 11, 2019
Merged

Adds a browse button next to the path text field for aux-based groups #4743

merged 6 commits into from
Mar 11, 2019

Conversation

samiyac
Copy link
Contributor

@samiyac samiyac commented Mar 10, 2019

Fixes #4586
Added a browse button for aux-based groups.


  • Change in CHANGELOG.md described
  • Tests created for changes
  • Manually tested changed features in running JabRef
  • Screenshots added in PR description (for bigger UI changes)
  • Ensured that the git commit message is a good one
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

@samiyac
Copy link
Contributor Author

samiyac commented Mar 10, 2019

I am relatively new to contributing to FOSS. Could one help me understand why the check failed and how I can resolve it?

Copy link
Member

@Siedlerchr Siedlerchr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution! The code looks so far, just some minor changes

@Siedlerchr
Copy link
Member

@samiyac

I am relatively new to contributing to FOSS. Could one help me understand why the check failed and how I can resolve it?

The failing check codecov refers to the Code Coverage, e..g how much of the code functionality is covered with tests. You can ignore it in most cases. We only tend to write tests for the most important parts (e.g. the classes in logic/model) and not for GUI dialogs.
Only if the Travis CI fails, you should check the details and see where and what error occurred, as this executes the checkstyle and the integration tests.

Copy link
Member

@tobiasdiez tobiasdiez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick follow-up. Code looks good to me now so I'll merge.

@tobiasdiez tobiasdiez merged commit fa984a7 into JabRef:master Mar 11, 2019
@samiyac samiyac deleted the fix-for-issue-4586 branch March 12, 2019 14:32
Siedlerchr added a commit that referenced this pull request Mar 16, 2019
…erbibtexkey

* upstream/master: (827 commits)
  [#4306] Disable renaming (#4727)
  Feature: implement search filter in show preferences (#4759)
  Enable import button only if entry selected (#4761)
  Improve Remote messaging (#4760)
  Add changelog entry for #4520
  Modifications to improve contrast of UI elements (#4758)
  Rework external changes dialog in JavaFX (#4693)
  Changes the title of Group Dialog to Add subgroup when adding a new subgroup (#4757)
  Optimize data fetching (#4520)
  Adds a browse button next to the path text field for aux-based groups (#4743)
  Saving changes and exiting application (#4729)
  Code optimized and created a reusable method to get writer output (#4750)
  Bump mvvmfx from 1.7.0 to 1.8.0 (#4747)
  Bump guava from 27.0.1-jre to 27.1-jre (#4748)
  Bump mvvmfx-validation from 1.7.0 to 1.8.0 (#4749)
  Remove obsolete swing components in Preferences and OpenOffice (#4740)
  Move library specific key pattern dialog call to library menu (#4744)
  Revert "Revert "Fix: bibkey generated does not handle diacritics" (#4741)" (#4742)
  Revert "Fix: bibkey generated does not handle diacritics" (#4741)
  Rename method to removeUnwantedCharacters
  ...
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