Skip to content

Conversation

mano-ranjan
Copy link
Collaborator

@mano-ranjan mano-ranjan commented Sep 1, 2025

πŸš€ Summary

Refactors NotificationModal to support customizable CTA label and action; updates PostAdvertInfo to use new props

🧩 Related Issues / Tasks

NA

πŸ› οΈ Type of Change

  • 🐞 Bug fix
  • ✨ New feature
  • 🧨 Breaking change
  • πŸ”¨ Enhancement/Refactor
  • πŸ“ Documentation
  • πŸ“œ Other (please describe):

βœ… Checklist

  • The code is formatted with Prettier (npm run format or npx prettier --write .)
  • PR references a linked Issue or Task number
  • Tested on both iOS and Android devices/emulators
  • No unnecessary logs or commented-out code
  • UI changes, if any, have been reviewed visually
  • All tests are passing (if applicable)
  • Added or updated documentation where necessary

πŸ“Έ Screenshots / Videos (UI changes only)

NA

πŸ§ͺ Test Plan

  1. Custom CTA Rendering – Verify that NotificationModal correctly displays different CTA labels passed via props.
  2. CTA Action Execution – Confirm that the CTA button triggers the provided action callback.
  3. PostAdvertInfo Integration – Ensure PostAdvertInfo passes the correct CTA props and that the modal behaves as expected in this flow.

@mano-ranjan mano-ranjan requested a review from leoAshu September 1, 2025 16:14
@mano-ranjan mano-ranjan self-assigned this Sep 1, 2025
@mano-ranjan mano-ranjan added the enhancement Refactor style or logic label Sep 1, 2025
@mano-ranjan mano-ranjan merged commit 2c0b8b4 into main Sep 1, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Refactor style or logic

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants