Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IS-2888: add møtebehov historikk #1607

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ingring
Copy link
Contributor

@ingring ingring commented Jan 16, 2025

Hva har blitt lagt til✨🌈

  • Lagt til møtebehov historikk på møtedialogsiden og i dialogmøte innkallingskjemaet.
  • Refaktorert litt propspropagering

Screenshots 📸✨

Skjermbilde 2025-01-16 kl  13 57 56
Skjermbilde 2025-01-16 kl  14 04 18
Skjermbilde 2025-01-16 kl  14 06 08

@ingring ingring requested a review from a team as a code owner January 16, 2025 13:05
@ingring ingring force-pushed the IS-2888-motebehov-historikk-dialogmoteside branch 3 times, most recently from a1fe4fa to 3c6e5b4 Compare January 16, 2025 13:38
@ingring ingring force-pushed the IS-2888-motebehov-historikk-dialogmoteside branch from 3c6e5b4 to 621c3eb Compare January 20, 2025 07:39
@ingring ingring force-pushed the IS-2888-motebehov-historikk-dialogmoteside branch 2 times, most recently from 973eabe to da757dc Compare January 21, 2025 08:36
@ingring
Copy link
Contributor Author

ingring commented Jan 21, 2025

Fått små innspill som går på brukerforståelsen av de tre ulike historikkboksene. For å tydeliggjøre forskjellen har vi endret ulike formuleringer. Og fjernet møtehistorikk ikonet siden vi ikke lenger bruker dette for å identifisere at det er historikk.

Skjermbilde 2025-01-21 kl  09 51 39

@@ -18,16 +18,11 @@ export function MoteSvarHistorikk({ historiskeMoter }: Props) {
const hasMoteHistorikk = historiskeMoter.length > 0;

return (
<Box background="surface-default" className="p-8">
<div className="flex flex-row mb-4">
<img src={FortidenImage} alt="moteikon" className="w-12 mr-4" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Vi kan nok fjerne denne FortidenImage helt hvis vi ikke bruker den lenger 👍🏼


const texts = {
header: "Møtesvarhistorikk",
header: "Svar på dialogmøte historikk",
Copy link
Contributor

Choose a reason for hiding this comment

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

Er Historikk dialogmøtesvar bedre? Føles litt som at det blir orddelingsfeil her, at det liksom er "Svar på dialogmøte"-historikk.
Men det er kanskje en bevisst greie fra Peter at vi kaller det "Svar på dialogmøte" i stedet for "Dialogmøtesvar"? 🤔 Tenker uansett vi kan dytte dette ut sånn som det står nå 👍🏼

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Godt innspill! Det er litt det samme Stine kommenterte på også, så endrer det bare tilbake til sånn det var.

@ingring ingring force-pushed the IS-2888-motebehov-historikk-dialogmoteside branch from da757dc to 6bb6ab0 Compare January 22, 2025 12:46
@ingring ingring force-pushed the IS-2888-motebehov-historikk-dialogmoteside branch from 6bb6ab0 to 35cb58a Compare January 22, 2025 12:47
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.

3 participants