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

feat: support half-yearly and yearly budget periods #1539

Merged
merged 2 commits into from
Mar 8, 2024

Conversation

clindeIntershop
Copy link
Contributor

@clindeIntershop clindeIntershop commented Nov 28, 2023

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:

What Is the Current Behavior?

It is not possible to select the budget periods 'half-yearly' or 'yearly' for cost center or user budgets.

Issue Number: Closes #

What Is the New Behavior?

The budget periods 'half-yearly' and 'yearly' are supported.

Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No

Other Information

requires ICM version 11.8/7.10.6/7.10.38.29-LTS or higher

AB91481

AB#91577

@SGrueber SGrueber self-requested a review November 30, 2023 12:44
marschmidt89
marschmidt89 previously approved these changes Nov 30, 2023
Copy link
Contributor

@marschmidt89 marschmidt89 left a comment

Choose a reason for hiding this comment

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

L10N changes approved.

@SGrueber SGrueber changed the title Bugfix/add costcenter budget periodes feat: support half-yearly and yearly budget periods Nov 30, 2023
@SGrueber SGrueber added the enhancement Enhancement to an existing feature label Nov 30, 2023
@SGrueber SGrueber added this to the 5.0 milestone Nov 30, 2023
@SGrueber SGrueber added the wait Waiting for something (e.g. new ICM release) label Nov 30, 2023
@SGrueber
Copy link
Collaborator

SGrueber commented Nov 30, 2023

@clindeIntershop If I try to change a user budget to 'half-yearly' I get a 400 http error:
image

image

image

Is there a certain ICM version needed for this feature? Please comment it at this pull request.

@clindeIntershop
Copy link
Contributor Author

@SGrueber yes you need a new (actually not released) ICM version that can handle this states. Where should i write this dependency so that the feature is not merged before the new ICM is released?

@shauke shauke modified the milestones: 5.0, 5.1 Dec 13, 2023
@SGrueber SGrueber force-pushed the bugfix/add-costcenter-budget-periodes branch from 9323643 to 433cfb8 Compare December 15, 2023 13:14
@SGrueber SGrueber removed the wait Waiting for something (e.g. new ICM release) label Mar 8, 2024
@SGrueber SGrueber force-pushed the bugfix/add-costcenter-budget-periodes branch 2 times, most recently from 6552465 to 7cb37f1 Compare March 8, 2024 12:51
@SGrueber SGrueber force-pushed the bugfix/add-costcenter-budget-periodes branch from 7cb37f1 to d347260 Compare March 8, 2024 12:57
@SGrueber SGrueber merged commit e2e7aea into develop Mar 8, 2024
21 checks passed
@SGrueber SGrueber deleted the bugfix/add-costcenter-budget-periodes branch March 8, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants