Skip to content

MCR-3672 Simplify default properties, unify default property naming stategy - #3082

Open
toKrause wants to merge 5 commits into
mainfrom
issues/MCR-3672_Simplify-default-properties-unify-default-property-naming-stategy
Open

toKrause wants to merge 5 commits into
mainfrom
issues/MCR-3672_Simplify-default-properties-unify-default-property-naming-stategy

Conversation

@toKrause

Copy link
Copy Markdown
Contributor

Link to jira.

Relies on #3075.

Pull Request Checklist (Author)

Please go through the following checklist before assigning the PR for review:

Ticket & Documentation

  • The issue in the ticket is clearly described and the solution is documented.
  • Design decisions (if any) are explained.
  • The ticket references the correct source and target branches.
  • The fixed-version is correctly set in the ticket and matches the PR's target branch (main).

Feature & Improvement Specific Checks

  • Instructions on how to test or use the feature are included or linked (e.g. to documentation).
  • For UI changes: before & after screenshots are attached.
  • New features or migrations are documented.
  • Does this change affect existing applications, data, or configurations?
    • Yes: Is a migration required? If yes, describe it.
    • Breaking change is marked in the commit message.

Testing

  • I have tested the changes locally.
  • The feature behaves as described in the ticket.
  • Were existing tests modified?
    • Yes: explain the changes for reviewers.

MCR Conventions & Metadata

  • MCR naming conventions are followed
  • If the public API has changed:
    • Old API is deprecated or a migration is documented.
    • If not, no action needed.
  • Java license headers are added where necessary.
  • Javadoc is written for non-self-explanatory classes/methods (Clean Code).
  • All configuration options are documented in Javadoc and mycore.properties.
  • No default properties are hardcoded — all set via mycore.properties.

Multi-Repo Considerations

  • Is an equivalent PR in MIR required?
    • If yes, is it already created?

@toKrause
toKrause force-pushed the issues/MCR-3672_Simplify-default-properties-unify-default-property-naming-stategy branch 5 times, most recently from e749d89 to 6034557 Compare August 1, 2026 12:56
@toKrause
toKrause force-pushed the issues/MCR-3672_Simplify-default-properties-unify-default-property-naming-stategy branch from 6034557 to 155032e Compare August 31, 2026 13:00
@toKrause
toKrause force-pushed the issues/MCR-3672_Simplify-default-properties-unify-default-property-naming-stategy branch 2 times, most recently from 86a4f52 to 37c9ddb Compare September 28, 2026 17:49
@toKrause
toKrause force-pushed the issues/MCR-3672_Simplify-default-properties-unify-default-property-naming-stategy branch from 37c9ddb to d311cca Compare October 26, 2026 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant