Skip to content

Add monthfiled to resolve strings fields #13379

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

Merged
merged 9 commits into from
Jun 21, 2025
Merged

Add monthfiled to resolve strings fields #13379

merged 9 commits into from
Jun 21, 2025

Conversation

Siedlerchr
Copy link
Member

@Siedlerchr Siedlerchr commented Jun 19, 2025

Fixes #13375

Steps to test

Open JabRef settings -> Entry -> Resolve strings for -> montfiled is present

Mandatory checks

  • I own the copyright of the code submitted and I license it under the MIT license
  • Change in CHANGELOG.md described in a way that is understandable for the average user (if change is visible to the user)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • [/ Screenshots added in PR description (if change is visible to the user)
  • [/] Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
  • [/] Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

@Siedlerchr Siedlerchr added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Jun 19, 2025
@koppor
Copy link
Member

koppor commented Jun 19, 2025

One more thing: Isnt monthfield an alias for month? Isn't it then an issue regarding aliassed fields?

@koppor
Copy link
Member

koppor commented Jun 19, 2025

One more thing: Isnt monthfield an alias for month? Isn't it then an issue regarding aliassed fields?

OMG, its filed; not field. Thus, no alias

@Siedlerchr
Copy link
Member Author

One more thing: Isnt monthfield an alias for month? Isn't it then an issue regarding aliassed fields?

OMG, its filed; not field. Thus, no alias

Yes! That's a tricky one. Had to carefully read the issue... 😆

@Siedlerchr Siedlerchr enabled auto-merge June 19, 2025 16:22
Copy link
Member

@koppor koppor left a comment

Choose a reason for hiding this comment

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

With nice preference migration. Thank you!

I removed duplicate "mothfiled" in 4f1e64a (#13379)

Copy link

trag-bot bot commented Jun 21, 2025

@trag-bot didn't find any issues in the code! ✅✨

@Siedlerchr Siedlerchr added this pull request to the merge queue Jun 21, 2025
Merged via the queue into main with commit c662e48 Jun 21, 2025
45 of 46 checks passed
@Siedlerchr Siedlerchr deleted the addMonthfield branch June 21, 2025 12:38
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

monthfiled formatted incorrectly in the BibTeX source
2 participants