feat send team emails

[johnvente]

Description

This allows you to send emails to specific teams in a course. The teams will be displayed as checkboxes, and the user will be able to select the teams as they wish.

Demo

demo.mp4

When teams are being fetched

---

Plugin loaded

How to test it

Setting up

You will need to install these following plugins:

• https://github.com/eduNEXT/platform-plugin-communications (lms)
• https://github.com/eduNEXT/platform-plugin-teams (cms)

If you are using devstack

1. Go to workspace and clone this branch

cd ~/workspace && git remote add fork [email protected]:eduNEXT/frontend-app-communications.git

git fetch fork jv/feat-send-team-emails-pluggable && git checkout jv/feat-send-team-emails-pluggable

2. Go to the mfe repository, use node version in the repo and install dependencies

cd ~/workspace/frontend-app-communications && nvm use && npm install

3. inside frontend-app-communications run

npm run start

4. Go to http://localhost:1984/courses/course-v1:edX+DemoX+Demo_Course/bulk_email Or
   http://localhost:1984/courses/{{id_course}}/bulk_email

5. Select any team or some teams in the field called "teams"

6. Fill out the fields Subject and Body

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a6dfabb) 83.31% compared to head (9752165) 83.31%.

Additional details and impacted files

@@                           Coverage Diff                            @@
##           jv/feat-send-indivudial-email-pluggable       #8   +/-   ##
========================================================================
  Coverage                                    83.31%   83.31%           
========================================================================
  Files                                           53       53           
  Lines                                          773      773           
  Branches                                       147      147           
========================================================================
  Hits                                           644      644           
  Misses                                         129      129           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[01001110J]

Is this still necessary?

[johnvente]

'no-unused-vars' isn't necessary, I will remove it. Thanks!

[01001110J]

I've seen this status in many different places, I would suggest to move into a variables, in order to avoid typos in the future.

Ej:

Suggested change

	const formStatusToast = ['error', 'complete', 'completeSchedule', 'loadingTeams'];
	const ERROR = 'error'
	const COMPLETE = 'complete'
	const COMPLETE_SCHEDULE = 'completeSchedule'
	const LOADING_TEAMS = 'loadingTeams'
	const formStatusToast = [ERROR, COMPLETE, COMPLETE_SCHEDULE, LOADING_TEAMS];

[01001110J]

Same about this states:

Suggested change

	isLoadingTeams ? 'loadingTeams' : '',
	COMPLETE_SCHEDULE,
	isLoadingTeams ? LOADING_TEAMS : '',

[01001110J]

Same here:

Suggested change

	state={isLoadingTeams ? 'loadingTeams' : formStatus}
	state={isLoadingTeams ? LOADING_TEAMS : formStatus}

[BryanttV]

Works as expected! 🚀