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.

Sign up for GitHub

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

Jump to bottom

Fix calendar add event 001 #331

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Fix calendar add event 001 #331

wants to merge 1 commit into from

Conversation

NTElissa
Copy link
Contributor

@NTElissa NTElissa commented Oct 21, 2023 •
edited
Loading

PR Description

This PR implements email notifications for invited users when new events are created. It also updates the email template used for inviting trainees to join events.

Description of Tasks Completed

  • Users can now be invited to events.
  • Invited users receive email notifications.
  • Invited users have the option to accept or reject invitations, along with providing a reason.
  • Once an invitation is confirmed, the event is added to everyone's calendar.

Functionality Added

  • Added the ability to invite trainees and, as a coordinator or admin, you can now invite them to calendar events.

How This Has Been Tested

  1. Cloned the repository using git clone fx-callendar-email-002.
  2. Installed necessary dependencies using npm install.
  3. Tested event invitations for trainees, ensuring proper functionality for different roles, such as coordinators or admins.

PR Checklist:

  • Updated the email template.
  • Tested the functionality with different roles.
  • (Optional) Additional tasks if applicable.

Track PR

Trello Link (#DP-?)

Screenshots (If Appropriate)

Before:

Screenshot Before

After:

Screenshot After

Screenshot from 2023-10-23 20-30-55

@vercel
Copy link

vercel bot commented Oct 21, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
metron-devpulse ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 28, 2023 7:58pm

Virgile1k
Virgile1k approved these changes Oct 24, 2023
View reviewed changes
Copy link

@Virgile1k Virgile1k left a comment

Choose a reason for hiding this comment

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

Checked and approved

ceelogre
ceelogre reviewed Oct 31, 2023
View reviewed changes
Copy link
Contributor

@ceelogre ceelogre left a comment

Choose a reason for hiding this comment

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

image

@ceelogre ceelogre changed the title Fix callendar add event 001 Fix calendar add event 001 Oct 31, 2023
@Dyves1 Dyves1 self-requested a review November 16, 2023 22:00
Oliviier-dev added a commit that referenced this pull request Oct 23, 2024
@Oliviier-dev
fix(#331): Improve invitation table pagination
05ff2b6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ceelogre ceelogre ceelogre left review comments

@KAYITARES KAYITARES KAYITARES approved these changes

@NdatimanaSamuel NdatimanaSamuel NdatimanaSamuel approved these changes

@Virgile1k Virgile1k Virgile1k approved these changes

@Dyves1 Dyves1 Dyves1 approved these changes

@rukundo-kevin rukundo-kevin Awaiting requested review from rukundo-kevin

@Gentille-dev Gentille-dev Awaiting requested review from Gentille-dev

@mken1205 mken1205 Awaiting requested review from mken1205

@nili-alu nili-alu Awaiting requested review from nili-alu

At least 3 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
Ready For Merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@NTElissa @ceelogre @KAYITARES @NdatimanaSamuel @Virgile1k @Dyves1 @Gentille-dev