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

bump fix notifications and mails send after voting #96

Merged
merged 1 commit into from
Aug 26, 2024

Conversation

BarbaraOliveira13
Copy link
Contributor

@BarbaraOliveira13 BarbaraOliveira13 commented Aug 23, 2024

🎩 Description
Bump ptp : Budget Booth, the vote confirmation is neither notified or sent by email.

📌 Related Issues
Notion card

Testing

Dashboard

  1. As an admin, go to a process.
  2. Go to the budget setting (in components, click on the settings icon).
  3. Click on “Vote in one: allows participants to vote in any budget, but only in one.” and “Voting enabled.”
Capture d’écran 2024-07-29 à 13 10 53 Capture d’écran 2024-07-29 à 15 45 36
  1. Update.

Front-end

  1. Go to the process you changed.
  2. Go to budgets.
  3. Start voting (enter the budget booth).
  4. Vote.
  5. Go to your notifications and confirm that you received one about your vote.
  6. Go to http://localhost:3000/letter_opener and confirm that you received an email about your vote.
Capture d’écran 2024-07-31 à 12 05 41

Copy link
Contributor

@Stef-Rousset Stef-Rousset left a comment

Choose a reason for hiding this comment

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

it works locally (after launching sidekiq in terminal); I received email and notification !!

@luciegrau luciegrau marked this pull request as ready for review August 26, 2024 13:22
@luciegrau luciegrau merged commit 2f7bb08 into develop Aug 26, 2024
14 checks passed
@luciegrau luciegrau deleted the fix/send_notification_budget_booth branch August 26, 2024 13:22
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