Skip to content

Add LocalisedChangeCasePropertyOutcome class#414

Merged
machacjozef merged 1 commit intorelease/7.0.0-rev10from
fix/localised_change_case_property_outcome
Feb 17, 2026
Merged

Add LocalisedChangeCasePropertyOutcome class#414
machacjozef merged 1 commit intorelease/7.0.0-rev10from
fix/localised_change_case_property_outcome

Conversation

@renczesstefan
Copy link
Member

@renczesstefan renczesstefan commented Feb 17, 2026

Description

Added missing localised outcome class.

Dependencies

No new dependencies were introduced

Third party dependencies

No new dependencies were introduced

Blocking Pull requests

There are no dependencies on other PR

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes have been checked, personally or remotely, with @machacjozef
  • I have commented my code, particularly in hard-to-understand areas
  • I have resolved all conflicts with the target branch of the PR
  • I have updated and synced my code with the target branch
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing tests pass locally with my changes:
    • Lint test
    • Unit tests
    • Integration tests
  • I have checked my contribution with code analysis tools:
  • I have made corresponding changes to the documentation:
    • Developer documentation
    • User Guides
    • Migration Guides

Summary by CodeRabbit

  • New Features
    • Added support for localized case property change event outcomes, improving multi-language response handling.

Introduce a new class to provide localized responses for ChangeCasePropertyOutcome events. This enhances internationalization support by extending LocalisedCaseEventOutcome with locale-specific handling.
@coderabbitai
Copy link

coderabbitai bot commented Feb 17, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

A new public class LocalisedChangeCasePropertyOutcome is introduced that extends LocalisedCaseEventOutcome. The class provides a constructor accepting a ChangeCasePropertyOutcome and a Locale parameter, which are delegated to the superclass constructor.

Changes

Cohort / File(s) Summary
New Localized Outcome Wrapper
application-engine/src/main/java/com/netgrif/application/engine/workflow/web/responsebodies/eventoutcomes/LocalisedChangeCasePropertyOutcome.java
New class extending LocalisedCaseEventOutcome with constructor that accepts ChangeCasePropertyOutcome and Locale parameters, delegating to superclass.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: adding a new LocalisedChangeCasePropertyOutcome class, which is the primary focus of the changeset.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@machacjozef machacjozef merged commit 431fa31 into release/7.0.0-rev10 Feb 17, 2026
6 of 7 checks passed
@machacjozef machacjozef deleted the fix/localised_change_case_property_outcome branch February 17, 2026 09:56
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.

3 participants

Comments