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

PN-13920 - Refactor redux and utility to handle new notification payments #1471

Merged
merged 9 commits into from
Feb 17, 2025

Conversation

alessandrogelmi
Copy link
Collaborator

@alessandrogelmi alessandrogelmi commented Feb 14, 2025

Short description

Refactor redux and utility to handle new notification payments

List of changes proposed in this pull request

  • Refactor initial state of newNotification slice
  • Refactor utility that map new notification from presentation to DTO layers

How to test

Check that redux tests and notification utility tests are ok

@AndreaCimini90 AndreaCimini90 self-requested a review February 14, 2025 15:22
@AndreaCimini90 AndreaCimini90 self-requested a review February 17, 2025 14:34
@AndreaCimini90 AndreaCimini90 merged commit 90d26c7 into feat/test-payments Feb 17, 2025
1 of 2 checks passed
@AndreaCimini90 AndreaCimini90 deleted the fix/PN-13920 branch February 17, 2025 16: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.

2 participants