Skip to content

feat: added notification UI#340

Merged
sundasnoreen12 merged 24 commits into
masterfrom
sundas/INF-820
Jun 15, 2023
Merged

feat: added notification UI#340
sundasnoreen12 merged 24 commits into
masterfrom
sundas/INF-820

Conversation

@sundasnoreen12

@sundasnoreen12 sundasnoreen12 commented May 22, 2023

Copy link
Copy Markdown
Contributor

INF-820

Description

Added Notification Button in Header Component.
Mockup for notifications tray and button: Notifications
Mockup for notifications button and variations: Notifications
The notifications tray is not scrollable. Implemented a load more button.
Added redux structure and binded the UI with the redux store

Screenshots
localhost_8080_

@sundasnoreen12 sundasnoreen12 changed the title Sundas/inf 820 feat: added notification view more notifications functionality May 22, 2023
@sundasnoreen12 sundasnoreen12 changed the title feat: added notification view more notifications functionality feat: added notification UI May 22, 2023
@sundasnoreen12
sundasnoreen12 force-pushed the sundas/INF-820 branch 3 times, most recently from 03377ca to 13545da Compare May 22, 2023 09:32
@codecov

codecov Bot commented May 22, 2023

Copy link
Copy Markdown

Codecov Report

Patch coverage: 23.63% and project coverage change: -18.54 ⚠️

Comparison is base (2543926) 61.87% compared to head (30e6eed) 43.33%.

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #340       +/-   ##
===========================================
- Coverage   61.87%   43.33%   -18.54%     
===========================================
  Files          15       28       +13     
  Lines         299      570      +271     
  Branches       70      108       +38     
===========================================
+ Hits          185      247       +62     
- Misses        112      310      +198     
- Partials        2       13       +11     
Impacted Files Coverage Δ
src/Notifications/data/hook.js 0.00% <0.00%> (ø)
src/common/time-locale.js 0.00% <0.00%> (ø)
src/Notifications/NotificationSections.jsx 4.34% <4.34%> (ø)
src/Notifications/index.jsx 4.34% <4.34%> (ø)
src/Notifications/NotificationTabs.jsx 6.66% <6.66%> (ø)
src/Notifications/utils.js 15.00% <15.00%> (ø)
src/Notifications/NotificationRowItem.jsx 16.66% <16.66%> (ø)
src/Notifications/data/api.js 16.66% <16.66%> (ø)
src/Notifications/data/slice.js 20.28% <20.28%> (ø)
src/Notifications/data/thunks.js 27.41% <27.41%> (ø)
... and 5 more

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Comment thread src/Notifications/icons/CheckCircleFilled.jsx Outdated
Comment thread src/index.scss
Comment thread src/constants.js Outdated
Comment thread src/time-locale.js
Comment thread src/Notifications/icons/index.js Outdated
Comment thread package.json Outdated
Comment thread package.json Outdated
Comment thread package.json Outdated
Comment thread package.json Outdated
Comment thread package.json Outdated
@sundasnoreen12
sundasnoreen12 force-pushed the sundas/INF-820 branch 2 times, most recently from 9d1e07c to a1d19b1 Compare May 30, 2023 06:35
Comment thread src/learning-header/AuthenticatedUserDropdown.jsx Outdated
Comment thread src/learning-header/AuthenticatedUserDropdown.jsx Outdated
Comment thread src/Notifications/data/selectors.js Outdated
Comment thread src/Notifications/data/selectors.js Outdated
Comment thread src/learning-header/AuthenticatedUserDropdown.jsx Outdated
Comment thread src/Notifications/data/slice.js Outdated
Comment thread src/Notifications/NotificationTabs.jsx Outdated
Comment thread src/Notifications/NotificationTabs.jsx Outdated
Comment thread src/Notifications/NotificationTabs.jsx Outdated
Comment thread src/Notifications/NotificationSections.jsx Outdated
Comment thread src/Notifications/data/selectors.js Outdated
Comment thread src/Notifications/data/slice.js Outdated
Comment thread src/Notifications/data/slice.js Outdated
Comment thread src/Notifications/data/slice.js Outdated
Comment thread src/Notifications/data/slice.js Outdated
Comment thread src/Notifications/NotificationTabs.jsx Outdated
Comment thread src/Notifications/NotificationSections.jsx Outdated
Comment thread src/Notifications/NotificationRowItem.jsx Outdated
Comment thread src/Notifications/NotificationRowItem.jsx Outdated
Comment thread src/Notifications/NotificationRowItem.jsx Outdated
@awais-ansari awais-ansari self-assigned this Jun 9, 2023
Comment thread src/Notifications/data/slice.js Outdated
Comment thread src/Notifications/data/slice.js
Comment thread src/Notifications/index.jsx Outdated
Comment thread src/Notifications/index.jsx Outdated
Comment thread src/Notifications/utils.js Outdated
Comment thread src/Notifications/data/thunks.js Outdated
Comment thread src/index.scss Outdated
Comment thread src/index.scss Outdated
Comment thread src/index.scss

@ayesha-waris ayesha-waris left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

minor nits otherwise PR LGTM
Please update screenshots before merging

@sundasnoreen12
sundasnoreen12 merged commit dad01fc into master Jun 15, 2023
@sundasnoreen12
sundasnoreen12 deleted the sundas/INF-820 branch June 15, 2023 08:12
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.

4 participants