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

fix: reportBy issue #2539 #2809

Closed
wants to merge 1 commit into from

Conversation

Rockingrajat
Copy link

@Rockingrajat Rockingrajat commented Oct 27, 2021

Fixes #2539

Changes proposed in this pull request:
reportBy now uses user.id instead of "some user"

Note: pull requests without proper descriptions may simply be closed without further discussion. We appreciate your contributions, but need to know what you are offering in clearly described format. Provide tests for all code that you add/modify. If you add/modify any components update the storybook. Thanks! (you can delete this text)

@vercel
Copy link

vercel bot commented Oct 27, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/hospitalrun/hospitalrun-frontend/8pyVkm8wZ7BiSGBfGNZvR1gp3kQP
✅ Preview: https://hospitalrun-frontend-git-fork-rockingrajat-master-hospitalrun.vercel.app

@CLAassistant
Copy link

CLAassistant commented Oct 27, 2021

CLA assistant check
All committers have signed the CLA.

@Rockingrajat Rockingrajat changed the title fixed the reportBy issue #2539 fix: reportBy issue #2539 Oct 27, 2021
@Rockingrajat Rockingrajat changed the title fix: reportBy issue #2539 feat: reportBy issue #2539 Oct 27, 2021
@Rockingrajat Rockingrajat changed the title feat: reportBy issue #2539 fix: reportBy issue #2539 Oct 27, 2021
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
2 participants