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

Display resource narrative #68

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Display resource narrative #68

wants to merge 7 commits into from

Conversation

BPierrick
Copy link
Contributor

@BPierrick BPierrick commented Jan 27, 2022

Fixes

Description

  • Displays Patient resource narrative in a Tooltip when hovering its Avatar
  • Displays common resource narratives inside a CardContent wrapper in their card.

Screenshot

Capture d’écran 2022-01-27 à 15 51 47

Definition of Done

  • I followed the Arkhn Code Book (I swear!).
  • I have written tests for the code I added or updated.
  • I have updated the documentation according to my changes.
  • I have updated the deployment configuration if needed.

@BPierrick BPierrick added enhancement New feature or request proj/vue-patient labels Jan 27, 2022
@BPierrick BPierrick requested review from MiskoG and a team January 27, 2022 14:53
Copy link

@simonvadee simonvadee left a comment

Choose a reason for hiding this comment

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

lgtm

innerHTML?: string;
};

const InnterHTMLDiv = ({

Choose a reason for hiding this comment

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

Suggested change
const InnterHTMLDiv = ({
const InnerHTMLDiv = ({

);
};

export default InnterHTMLDiv;

Choose a reason for hiding this comment

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

Suggested change
export default InnterHTMLDiv;
export default InnerHTMLDiv;

@@ -9,6 +9,8 @@ import { DateTime } from "luxon";
import { useTranslation } from "react-i18next";
import { useParams } from "react-router-dom";

import InnterHTMLDiv from "common/components/InnerHTMLDiv";

Choose a reason for hiding this comment

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

Inner !


import { CardContent } from "@mui/material";

import InnterHTMLDiv from "common/components/InnerHTMLDiv";

Choose a reason for hiding this comment

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

Inner !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request proj/vue-patient
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Display Ressource narrative if present
3 participants