Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

fix: incident reportedBy from currently logged in user ID #2988

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

xhefribala
Copy link

@xhefribala xhefribala commented Sep 19, 2022

Fixes #2539

Changes proposed in this pull request:

Access current userLoginID with Redux useSelector Hook (to fix issue #2539)
Get fullName user associated with userLoginID and show in 'reportedBy' in the report incident file.
Changed version lodash "4.17.15" to version "4.14.182" (to fix npm start issues)
Changed version chalk "^5.0.0" to version "~4.1.2" (to fix npm start issues)
Changed version types/node "17.0.0", to version "~15.6.1" (to fix npm start issues)
After fix:
Screen Shot 2022-09-19 at 10 24 21 am

@vercel
Copy link

vercel bot commented Sep 19, 2022

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

Name Status Preview Updated
hospitalrun-frontend ❌ Failed (Inspect) Sep 19, 2022 at 4:30AM (UTC)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incident - reportedBy should store the currently logged in user id
1 participant