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.

Sign up for GitHub

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

Jump to bottom

feat: properties #932

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

feat: properties #932

wants to merge 8 commits into from

Conversation

MaxSchaefer
Copy link
Member

No description provided.

@DasProffi @MaxSchaefer
feat: add property details (#904)
9b4aa19 
* feat: add property details

* feat: add property display

* feat update search documentation

* feat update search documentation

* chore: update key in table header

Co-authored-by: Max Schäfer <[email protected]>

* chore: update key in table header

Co-authored-by: Max Schäfer <[email protected]>

* chore: update key in table header

Co-authored-by: Max Schäfer <[email protected]>

* chore: strongly typed translations

* feat: add archive button

* feat: add archive modal

* chore: stronger translation typing

* chore: soft-required with checkbox

* chore: empty values use placeholder

---------

Co-authored-by: Max Schäfer <[email protected]>
@cloudflare-pages Cloudflare Pages
Copy link

cloudflare-pages bot commented Mar 15, 2024 •
edited
Loading

Deploying web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9c8ec6c
Status: ✅  Deploy successful!
Preview URL: https://255247d9.web-d91.pages.dev
Branch Preview URL: https://integration-properties.web-d91.pages.dev

View logs

@github-actions GitHub Actions
Copy link

Preview of tasks

https://pr-932-helpwave-web-tasks.fly.dev

DasProffi and others added 4 commits March 17, 2024 13:19
@DasProffi @MaxSchaefer
feat: add create property form (#931)
4fd9a30 
* chore: add stepper to create property

* chore: add stepper component

* feat: update stepping

* chore: merge main and fix translations

* feat: move subjectType to context

---------

Co-authored-by: Max Schäfer <[email protected]>
@MaxSchaefer
Merge branch 'main' into integration/properties
a8fc284
@DasProffi @MaxSchaefer
feat: add propertyList (#935)
88b8f4a 
* feat: add PropertyList

* feat: add Checkbox to PropertyList

* feat: update translations

* feat: add property list to PatientDetails

* feat: add mutations with backend mocking

* feat: add mutations to patient list

* feat: add save behavior to properties

* feat: add search to add property

* chore: remove console.log

* fix: fix soft required not marking

* feat: add property details pseudo backend connection

* fix: fix wrong id in example data

* fix: fix usage of wrong method

Co-authored-by: Max Schäfer <[email protected]>

* feat: use Memo for caching and fix updating properties

---------

Co-authored-by: Max Schäfer <[email protected]>
@DasProffi
feat: date property without time (#945)
079e14e 
* feat: add date and dateTime distinction

* fix: fix translations

* fix: fix translation

* chore: update parameters of Input onChange and editComplete
@jonas-ester
Copy link

jonas-ester commented Apr 8, 2024 •
edited
Loading

DESIGN QA
Anzeigefehler beim Remove von required Feldern
Bildschirmfoto 2024-04-08 um 18 15 32

@jonas-ester
Copy link

jonas-ester commented Apr 8, 2024 •
edited
Loading

DESIGN QA

"Remove" Button nur im Hover State anzeigen

Aktion
//

Ergebnis
Der "Remove" Button ist dauerhaft sichtbar

Erwartetes Ergebnis
Der "Remove" Button ist nur sichtbar wenn der Cursor über dem entsprechenden Property steht.

@jonas-ester
Copy link

DESIGN QA

Entfernen von non-required Feldern.

Aktion
Klicken auf den "Remove" Button eines Properties

Ergebnis
Der Inhalt des Feldes wird geleert, das Feld bleibt bestehen

Erwartetes Ergebnis
Der Inhalt des Feldes wird geleert und das Feld verschwindet

@jonas-ester
Copy link

DESIGN QA

Beim Angelgen von Properties bitte den Page Indikator entfernen:
image

@jonas-ester
Copy link

DESIGN QA

Erstellen von Properties: bereits ausgefüllte Sektionen sollen sich wieder öffnen lassen.

Aktion
Klicken auf den Button "Next Step"

Ergebnis
Die nächste Sektion wird aktiv und expanded und die vorherige collapsed und wird inaktiv

Erwartetes Ergebnis
Die nächste Sektion wird aktiv und expanded und die vorherige collapsed, bleibt aber aktiv und kann aufgeklappt werden.

@jonas-ester
Copy link

DESIGN QA

Hinzufügen von Values zu einem Multi Select

Aktion

  1. Erstellen eines neuen Property
  2. Als "Field Type" Singel Select oder Multi Select wählen

Ergebnis
Unter "Values" werden direkt mehrere leere Felder angezeigt und es lassen sich über den kleinen + Button weitere hinzufügen.

Erwartetes Ergebnis
Es gibt immer ein Leeres Value-Feld, dass sich nicht löschen lässt. Sobald es einen Inhalt hat wird ein neues leeres Feld unten an die Liste angefügt.

image

@jonas-ester
Copy link

DESIGN QA

Liste verfügbarer Properties

Aktion
Aufrufen der Property Übersicht

Ergebnis
Alle verfügbaren Properties werden angezeigt

Erwartetes Ergebnis
Bitte das Layout der einzelnen Properties List Items an das in Figma angleichen. Für eine intuitive Bedienung ist hier vor allem der Selected State wichtig.
Bildschirmfoto 2024-04-09 um 18 22 51

@jonas-ester
Copy link

DESIGN QA

Filtern verfügbarer Properties

Aktion

  1. Aufrufen der Property Übersicht
  2. Auswählen eines Subject Filters

Ergebnis

  • Es ist dauerhaft ein "Remove Filter" Button sichtbar
  • Das Suchfeld passt seine Breite an die Breite des Filters an, wodurch das Filter Element nach links oder rechts springt.

Erwartetes Ergebnis

  • Es gibt keinen "Remove Filter" Button
  • Der Property Filter bekommt eine andere Farbe
  • Das Dropdown Icon auf der rechten Seite wird zu einem X
  • Klicken auf das X entfernt den Filter und zeigt wieder das anfängliche Design
  • Das Suchfeld behält eine konstante Breite
Bildschirmfoto 2024-04-09 um 18 23 22

@MaxSchaefer
Merge main into integration/properties via merge-main-into-integration (
9c8ec6c 
#1040)

* Merge main into merge-main-into-integration-properties

* fix: remove duplicate Tile story

* fix: layout issue for properties

* fix: bg-transparent on language-switcher
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
DO NOT MERGE
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@MaxSchaefer @jonas-ester @DasProffi