cb09850f-bb9d-4a14-8534-f182f52730b3

Committed 10 Jul 2024 08:54AM UTC coverage: 92.973% (-0.05%) from 93.026%

Build # cb09850f-bb9d-4a14-8534-f182f52730b3

Build Type

Pull #42

circleci

Committed by

Yvan-David

Commit Message

feat(notifications): Implement mark all Notifications as Read

- On click the button the notification pane should be empty
- The notifications pane should open up when a user clicks on the icon
- User can only see unread notifications

[Delivers #187364710]

Pull Request Pull Request #42: [#187364710] Marking all notifications as read

Run Details

405 of 481 branches covered (84.2%)

Branch coverage included in aggregate %.

17 of 17 new or added lines in 2 files covered. (100.0%)

1 existing line in 1 file now uncovered.

1143 of 1184 relevant lines covered (96.54%)

10.23 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

92.42

/src/views/NotificationPane.tsx

import { useEffect, useState } from "react";
import axiosClient from "../hooks/AxiosInstance";
import { toast } from "react-toastify";
import Modal from "react-modal";
import { FaTimes } from "react-icons/fa";
import noNotification from "../assets/images/no-notification.jpg";

//Modal.setAppElement('#root');
interface Notification {
  id: number;
  productId: string;
  userId: string;
  message: string;
  isRead: boolean;
  createdAt: string
}

interface ModalProps {
  open: boolean
  onClose: () => void;
}



const NotificationPane: React.FC<ModalProps> = ({ open, onClose }) => {
  const [notifications, setNotifications] = useState<Notification[]>([]);
  const [page, setPage] = useState(1);
  const [totalPages, setTotalPages] = useState(1);
  const [isLoading, setIsLoading] = useState<boolean>(false);
  const [isModalOpen, setIsModalOpen] = useState(false);
  const [allRead, setAllRead] = useState(false);
  const client = axiosClient();
  const notify = (message: string) => toast(message);

  const fetchNotifications = async (pageNumber: number) => {
    setIsLoading(true);
    try {
      const response = await client.get(`/notifications?page=${pageNumber}`);
      setNotifications(response.data.notifications);
      setTotalPages(response.data.pagination.totalPages);
      setAllRead(notifications.every((n: Notification) => n.isRead));
      console.log(response)
    } catch (err: any) {
      notify("failed to fetch notifications");
    } finally {
      setIsLoading(false);
    }
  };
  const markAllAsRead = async () => {
    try {
      const response = await client.put("/notifications/all");
      if(response.status === 200){
        notify("All notifications marked as read");
        setAllRead(true)
      }
    } catch (err: any) {

    }
  };

  useEffect(() => {
    fetchNotifications(page);
  }, [page]);
  useEffect(() => {
    if (open) {
      setIsModalOpen(true);
    }
  }, [open]);

  const handleNextPage = () => {
    if (page < totalPages) {
      setPage(page + 1);
    }
  };

  const handlePrevPage = () => {
    if (page > 1) {
      setPage(page - 1);
    }
  };

  const closeModal = () => {
    setIsModalOpen(false);
    onClose()
  };
  return (
    <div className="relative p-4">
      <Modal
        isOpen={isModalOpen}
        onRequestClose={closeModal}
        contentLabel="Notifications"
        className="absolute top-0 left-0 right-0 bottom-0 m-auto bg-white p-8 rounded shadow-lg w-3/4 h-3/4 max-h-full overflow-auto"
        overlayClassName="fixed inset-0 bg-black bg-opacity-50"
      >
        <div className="flex flex-col h-full p-4 bg-chat_back">
          <h2 className="text-xl font-semibold mb-4">Notifications</h2>
          <div className="flex-1 overflow-y-auto rounded p-4">
            <ul className="space-y-4">
              {isLoading ? (
                <li className="text-center py-4">Loading...</li>
              ) : allRead ? (
                <li className="flex justify-center items-center py-4"><img src={noNotification} alt="No new notification" className="mx-auto" /></li>
              ): (
                notifications.map((notification, index) => (
                  <li
                    key={notification.id}
                    className={`p-3 rounded-lg flex items-start space-x-2 ${index % 2 === 0 ? 'bg-grayboro' : 'bg-gray-500'}`}
                  >
                    <div className="flex-1">
                      <div className="flex justify-between mb-1">
                        <span className="text-custom-size text-gray-500 self-end">
                          {new Date(notification.createdAt).toLocaleDateString()}{" "}
                          {new Date(notification.createdAt).toLocaleTimeString()}
                        </span>
                      </div>
                      <p className={`text-sm ${notification.isRead ? 'font-normal' : 'font-bold'}`}>
                        {notification.message}
                      </p>
                    </div>
                  </li>
                ))
              )}
            </ul>
          </div>
          <div className="flex justify-between mt-4">
            <button
              onClick={handlePrevPage}
              className={`bg-back_next text-white px-4 py-2 rounded ${page === 1 && "opacity-50 cursor-not-allowed"}`}
              disabled={page === 1}
              name='Back'
            >
              Back
            </button>
            <h4>Page {page}</h4>
            <button
              onClick={handleNextPage}
              className={`bg-back_next text-white px-4 py-2 rounded ${page === totalPages && "opacity-50 cursor-not-allowed"}`}
              disabled={page === totalPages}
               name='Next'
            >
              Next
            </button>
          </div>

          {!allRead && (
            <div className="flex justify-end mt-4">
              <button
                onClick={markAllAsRead}
                className="bg-green-800 text-white px-4 py-2 rounded"
                 name='Mark All as Read'
              >
                Mark All as Read
              </button>
            </div>
          )}
          <div className="absolute top-2 right-2 px-4 py-2 rounded">
            <FaTimes size={22} onClick={closeModal}  name='Close' data-testid="close-icon"/> 
          </div >
         
        </div>
      </Modal>
    </div>
  );
};

export default NotificationPane;

1	import { useEffect, useState } from "react";	5✔
2	import axiosClient from "../hooks/AxiosInstance";	5✔
3	import { toast } from "react-toastify";	5✔
4	import Modal from "react-modal";	5✔
5	import { FaTimes } from "react-icons/fa";	5✔
6	import noNotification from "../assets/images/no-notification.jpg";	5✔
7
8	//Modal.setAppElement('#root');
9	interface Notification {
10	id: number;
11	productId: string;
12	userId: string;
13	message: string;
14	isRead: boolean;
15	createdAt: string
16	}
17
18	interface ModalProps {
19	open: boolean
20	onClose: () => void;
21	}
22
23
24
25	const NotificationPane: React.FC<ModalProps> = ({ open, onClose }) => {	5✔
26	const [notifications, setNotifications] = useState<Notification[]>([]);	114✔
27	const [page, setPage] = useState(1);	114✔
28	const [totalPages, setTotalPages] = useState(1);	114✔
29	const [isLoading, setIsLoading] = useState<boolean>(false);	114✔
30	const [isModalOpen, setIsModalOpen] = useState(false);	114✔
31	const [allRead, setAllRead] = useState(false);	114✔
32	const client = axiosClient();	114✔
33	const notify = (message: string) => toast(message);	114✔
34
35	const fetchNotifications = async (pageNumber: number) => {	114✔
36	setIsLoading(true);	45✔
37	try {	45✔
38	const response = await client.get(`/notifications?page=${pageNumber}`);	45✔
39	setNotifications(response.data.notifications);	16✔
40	setTotalPages(response.data.pagination.totalPages);	14✔
41	setAllRead(notifications.every((n: Notification) => n.isRead));	14✔
42	console.log(response)	14✔
43	} catch (err: any) {
44	notify("failed to fetch notifications");	31✔
45	} finally {
46	setIsLoading(false);	45✔
47	}
48	};
49	const markAllAsRead = async () => {	114✔
50	try {	2✔
51	const response = await client.put("/notifications/all");	2✔
52	if(response.status === 200){	2✔
53	notify("All notifications marked as read");	2✔
54	setAllRead(true)	2✔
55	}
56	} catch (err: any) {
57
58	}
59	};
60
61	useEffect(() => {	114✔
62	fetchNotifications(page);	45✔
63	}, [page]);
64	useEffect(() => {	114✔
65	if (open) {	44✔
66	setIsModalOpen(true);	16✔
67	}
68	}, [open]);
69
70	const handleNextPage = () => {	114✔
71	if (page < totalPages) {	2✔
72	setPage(page + 1);	2✔
73	}
74	};
75
76	const handlePrevPage = () => {	114✔
77	if (page > 1) {	1✔
78	setPage(page - 1);	1✔
79	}
80	};
81
82	const closeModal = () => {	114✔
83	setIsModalOpen(false);	1✔
84	onClose()	1✔
85	};
86	return (	114✔
87	<div className="relative p-4">
88	<Modal
89	isOpen={isModalOpen}
90	onRequestClose={closeModal}
91	contentLabel="Notifications"
92	className="absolute top-0 left-0 right-0 bottom-0 m-auto bg-white p-8 rounded shadow-lg w-3/4 h-3/4 max-h-full overflow-auto"
93	overlayClassName="fixed inset-0 bg-black bg-opacity-50"
94	>
95	<div className="flex flex-col h-full p-4 bg-chat_back">
96	<h2 className="text-xl font-semibold mb-4">Notifications</h2>
97	<div className="flex-1 overflow-y-auto rounded p-4">
98	<ul className="space-y-4">
99	{isLoading ? (	114✔
100	<li className="text-center py-4">Loading...</li>
101	) : allRead ? (	80✔
102	<li className="flex justify-center items-center py-4"><img src={noNotification} alt="No new notification" className="mx-auto" /></li>
103	): (
104	notifications.map((notification, index) => (
UNCOV 105	<li	×
106	key={notification.id}
107	className={`p-3 rounded-lg flex items-start space-x-2 ${index % 2 === 0 ? 'bg-grayboro' : 'bg-gray-500'}`}	×
108	>
109	<div className="flex-1">
110	<div className="flex justify-between mb-1">
111	<span className="text-custom-size text-gray-500 self-end">
112	{new Date(notification.createdAt).toLocaleDateString()}{" "}
113	{new Date(notification.createdAt).toLocaleTimeString()}
114	</span>
115	</div>
116	<p className={`text-sm ${notification.isRead ? 'font-normal' : 'font-bold'}`}>	×
117	{notification.message}
118	</p>
119	</div>
120	</li>
121	))
122	)}
123	</ul>
124	</div>
125	<div className="flex justify-between mt-4">
126	<button
127	onClick={handlePrevPage}
128	className={`bg-back_next text-white px-4 py-2 rounded ${page === 1 && "opacity-50 cursor-not-allowed"}`}	222✔
129	disabled={page === 1}
130	name='Back'
131	>
132	Back
133	</button>
134	<h4>Page {page}</h4>
135	<button
136	onClick={handleNextPage}
137	className={`bg-back_next text-white px-4 py-2 rounded ${page === totalPages && "opacity-50 cursor-not-allowed"}`}	221✔
138	disabled={page === totalPages}
139	name='Next'
140	>
141	Next
142	</button>
143	</div>
144
145	{!allRead && (	215✔
146	<div className="flex justify-end mt-4">
147	<button
148	onClick={markAllAsRead}
149	className="bg-green-800 text-white px-4 py-2 rounded"
150	name='Mark All as Read'
151	>
152	Mark All as Read
153	</button>
154	</div>
155	)}
156	<div className="absolute top-2 right-2 px-4 py-2 rounded">
157	<FaTimes size={22} onClick={closeModal} name='Close' data-testid="close-icon"/>
158	</div >
159
160	</div>
161	</Modal>
162	</div>
163	);
164	};
165
166	export default NotificationPane;	5✔

atlp-rwanda / e-commerce-bitcrafters-fn / cb09850f-bb9d-4a14-8534-f182f52730b3

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous