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 respecting overWriteOwner preferences #9896

Merged
merged 3 commits into from
May 15, 2023
Merged

Conversation

koppor
Copy link
Member

@koppor koppor commented May 15, 2023

I removed some Booleans from the code. I think, the "overWriteOwner" preference is now working. Before, it was always overwritten.

Compulsory checks

@koppor koppor added the type: code-quality Issues related to code or architecture decisions label May 15, 2023
Siedlerchr
Siedlerchr previously approved these changes May 15, 2023
@Siedlerchr Siedlerchr added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label May 15, 2023
@calixtus calixtus merged commit 9b3816d into JabRef:main May 15, 2023
@calixtus calixtus deleted the fix-overwrite branch May 15, 2023 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers type: code-quality Issues related to code or architecture decisions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants