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

feat/add data model binding for subfrom #13928

Closed
JamalAlabdullah opened this issue Oct 28, 2024 · 9 comments · Fixed by #13961
Closed

feat/add data model binding for subfrom #13928

JamalAlabdullah opened this issue Oct 28, 2024 · 9 comments · Fixed by #13961
Assignees
Labels
added-to-sprint frontend team/studio-domain2 text/content used for issues that need som text improvements, often by ux

Comments

@JamalAlabdullah
Copy link
Contributor

JamalAlabdullah commented Oct 28, 2024

This issue depends on the scenario where, when creating a new subform ("Opprett et nytt skjema"), a new card will be shown to enter the name of the subform and choose the data model to bind to the subform.

User Story

As a user, I want to be able to click on an "Opprett et nytt underskjema" ("Create a new sub-form") button directly from the layout set selection panel, so that I can create a new layout set for a subform and then choose the data model to bind to the subform.

How

After the user navigates to the panel with the dropdown menu for layout sets, we should add a dropdown that displays all available data models, allowing the user to select one to bind to the subform as Figma shows.

@JamalAlabdullah JamalAlabdullah converted this from a draft issue Oct 28, 2024
@JamalAlabdullah JamalAlabdullah added added-to-sprint frontend team/studio-domain2 text/content used for issues that need som text improvements, often by ux labels Oct 28, 2024
@JamalAlabdullah JamalAlabdullah self-assigned this Oct 28, 2024
@lassopicasso lassopicasso moved this from ⚠️ Blocked to 📈 Todo in Team Studio Oct 29, 2024
@JamalAlabdullah JamalAlabdullah linked a pull request Oct 31, 2024 that will close this issue
4 tasks
@JamalAlabdullah JamalAlabdullah removed their assignment Oct 31, 2024
@nkylstad
Copy link
Member

nkylstad commented Nov 4, 2024

@JamalAlabdullah is this not a duplicate of #13764?

@nkylstad No, in #13928 we add select for datamodel and functionalty, but in #13764 will hide the selector for data model as it is described here :
#13764 (comment)

@mlqn mlqn assigned mlqn and JamalAlabdullah and unassigned mlqn Nov 4, 2024
@Ildest
Copy link
Contributor

Ildest commented Nov 5, 2024

Reviewed: @JamalAlabdullah I think this is the correct Figma design/text, but we will check with @Annikenkbrathen as soon as she is back: Image

@Annikenkbrathen
Copy link

Annikenkbrathen commented Nov 5, 2024

This is the right design @JamalAlabdullah. The text has been slightly changed.

Image

If I understood this issue correctly, we are missing the design of how it should look and function when pressing the "+ opprett et nytt underskjema" button. I’ve tried to illustrate it with arrows to show how the view will look.

If you choose to create a new one, you shouldn’t be able to select both the new and an existing subform when adding a subform. Therefore, I suggest that the option to select an existing one disappears when you click the add button. And if you delete the add subform card, you return to the initial view.

@JamalAlabdullah
Copy link
Contributor Author

JamalAlabdullah commented Nov 6, 2024

This is the right design @JamalAlabdullah. The text has been slightly changed.

Image

If I understood this issue correctly, we are missing the design of how it should look and function when pressing the "+ opprett et nytt underskjema" button. I’ve tried to illustrate it with arrows to show how the view will look.

If you choose to create a new one, you shouldn’t be able to select both the new and an existing subform when adding a subform. Therefore, I suggest that the option to select an existing one disappears when you click the add button. And if you delete the add subform card, you return to the initial view.

@Annikenkbrathen I did not get information about this user story you talk about 😢 and i get info that we do not need delete button. Do we need also add button in both cards? because in our implementation now when the user choose a subform from selector then the card lucks and the name is saved.
one more thing the text "Opprett et nytt undersjema" was the first text we used but i get feedback to change it to "Lag ..." so can we say "Opprett et nytt undersjema" is the last updated one?
If we will go for these changes then maybe we should fix in a separate issue because this PR is just to add data model.
what do you think @framitdavid ?

@Annikenkbrathen
Copy link

Annikenkbrathen commented Nov 6, 2024

Slik som designet er her, så må du åpne en card-visning når du trykker + opprett et nytt underskjema. Det ser kanskje merkelig ut om det kortet legger seg under nedtrekken "velg underskjema", så derfor foreslo jeg at vi bytter ut visning på bilde 1, med et card.
Da må jo brukeren får mulighet til å angre, så jeg la til "slett" -knappen igjen.

Om du trykker på slett knappen. så går du bare tilbake til forrige visning hvor du kan velge et eksiterende underskjema.

@Annikenkbrathen Ok, Jeg skal lage en ny issue på det.

@lassopicasso
Copy link
Contributor

lassopicasso commented Nov 7, 2024

Jeg tok fatt på det nye issuet som lå på todo som du laget om dette, @JamalAlabdullah . (Håper det var greit! 🙂)
Joiner derfor denne diskusjonen. Forstår jeg det riktig at den lagre-knappen under nedtrekkslisten skal være synlig - kortet lagrer og lukker seg først når bruker har klikket på den, @Annikenkbrathen? Slik det er nå lagres og lukkes automatisk når bruker har valgt i nedtrekkslisten og da trengs det ikke en knapp der.

Også lurer jeg på om det er fortsatt skal være slik at man får dette varselet i konfigen hvis det ikke finnes noe underskjema, bilde 1?
Eller skulle bruker også få mulighet til å opprette underskjema slik som bilde nr 2, men kanskje uten slett-knappen i det tilfellet?

Bilde 1:
Image

Bilde 2:
Image

@JamalAlabdullah
Copy link
Contributor Author

@lassopicasso yes 🙂 den er klar for å starte med.
jeg er enig med første delen du nevnt, at nedtrekkslisten lukker og lager, så har har vi kanskje 2 muligheter:

  • Legge til lagre knappen men la lagring prossess skjer gjennom knappen og nedtrekkslisten bare for å velge underskjema.
  • Eller la den som den er nå.
    Men jeg tror Anniken mente at vi skal gjøre som bildet viser med knapp.

@Annikenkbrathen
Copy link

Ja, skjønner poengene deres. Det eneste er jo at den går rett fra dette recomennededNext steget til konfigpanelet om du velger underskjema fra nedtrekkslisten.

Så hvis du hadde hadde en lagre knapp der, så kan brukeren aktivt velge når de vil lagre og lukke det recomennededNet cardet, eller bytte til et annet underskjema i listen.

Men i og med at man kan endre underskjema senere fra konfigpaneletet så er det ikke sikkert vi trenger den knappen nei. Det kan være det bare blir masete med et ekstra klikk. Så er enig i at det kan være en god ide å prøve å droppe lagreknappen på første skisse (med nedtrekken).

@Annikenkbrathen
Copy link

Også lurer jeg på om det er fortsatt skal være slik at man får dette varselet i konfigen hvis det ikke finnes noe underskjema, bilde 1?

@lassopicasso
Nei, nå vil jo ikke dette være en case lengre. For når du drar inn "tabell for underskjema" - komponenten, så kommer brukeren rett til recommended next kortet, og blir bedt om å lage et underskjema og utforme det. Denne tror jeg noen allerede har lagt inn?

Image

@mlqn mlqn removed their assignment Nov 7, 2024
@github-project-automation github-project-automation bot moved this from 🧪 Test to Documentation in Team Studio Nov 8, 2024
@lassopicasso lassopicasso moved this from Documentation to ✅ Done in Team Studio Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
added-to-sprint frontend team/studio-domain2 text/content used for issues that need som text improvements, often by ux
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

6 participants