Skip to content

Add config to hide voucher amount on webshop#2818

Merged
RobinMeles merged 2 commits intodevelopfrom
feature.add-config-to-hide-voucher-amount-on-webshop
Mar 18, 2026
Merged

Add config to hide voucher amount on webshop#2818
RobinMeles merged 2 commits intodevelopfrom
feature.add-config-to-hide-voucher-amount-on-webshop

Conversation

@ateamcms
Copy link
Contributor

@ateamcms ateamcms commented Mar 5, 2026

Changes description

Developers checklist

  • Autotests are passed locally
  • Migration rollback works - if there is migration, check rollback
  • Autotests are added:
    • bugfix - unit/feature test for this scenario if possible
    • new small/medium feature - simple feature test for positive scenarios

QA checklist

  • Translations are done
  • Endpoint is fast on dump - if there is new endpoint or changed query, endpoints that are using changed query - working fast on production dump, <2s

@ateamcms ateamcms requested a review from dev-rminds March 5, 2026 15:33
@dev-rminds dev-rminds requested a review from RobinMeles March 9, 2026 12:33
@dev-rminds dev-rminds assigned RobinMeles and unassigned dev-rminds Mar 9, 2026
@dev-rminds dev-rminds marked this pull request as ready for review March 9, 2026 12:34
@RobinMeles
Copy link
Contributor

@ateamcms could you maybe check and fix the merge conflict?

@RobinMeles RobinMeles assigned ateamcms and unassigned RobinMeles Mar 9, 2026
…-to-hide-voucher-amount-on-webshop

# Conflicts:
#	app/Http/Resources/VoucherResource.php
@ateamcms
Copy link
Contributor Author

ateamcms commented Mar 9, 2026

@RobinMeles done

@RobinMeles RobinMeles assigned RobinMeles and unassigned ateamcms Mar 9, 2026
@RobinMeles RobinMeles merged commit 355136c into develop Mar 18, 2026
6 checks passed
@RobinMeles RobinMeles deleted the feature.add-config-to-hide-voucher-amount-on-webshop branch March 18, 2026 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants