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

Fix user login #2987

Closed
wants to merge 16 commits into from
Closed

Fix user login #2987

wants to merge 16 commits into from

Conversation

xhefribala
Copy link

@xhefribala xhefribala commented Sep 13, 2022

Fixes #2539

Changes proposed in this pull request:

  • Access current userLoginID with Redux useSelector Hook (to fix issue Incident - reportedBy should store the currently logged in user id #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-14 at 10 06 53 am
Screen Shot 2022-09-14 at 10 06 58 am

Newly added dependencies with Bundlephobia links:

No new dependencies added

@vercel
Copy link

vercel bot commented Sep 13, 2022

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

Name Status Preview Updated
hospitalrun-frontend ✅ Ready (Inspect) Visit Preview Sep 13, 2022 at 4:00AM (UTC)

@CLAassistant
Copy link

CLAassistant commented Sep 13, 2022

CLA assistant check
All committers have signed the CLA.

@matteovivona
Copy link
Contributor

@xhefribala please fix commits message using a semantic version format “fix(login): some message”

@xhefribala xhefribala closed this Sep 19, 2022
@xhefribala xhefribala deleted the fixUserLogin branch September 19, 2022 00:21
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
4 participants