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

Implementation: As a user I want to set a start date of one-time report #3753

Open
RomijnHumanoids opened this issue Oct 29, 2024 · 2 comments · May be fixed by #3840
Open

Implementation: As a user I want to set a start date of one-time report #3753

RomijnHumanoids opened this issue Oct 29, 2024 · 2 comments · May be fixed by #3840
Assignees
@Rieven
Labels
approved for implementation (design done) frontend 😸 Review/QA feedback Review/QA feedback provided

Comments

@RomijnHumanoids
Copy link

RomijnHumanoids commented Oct 29, 2024 •
edited
Loading

In the current implementation (29 oct.) users are able to set the start date of scheduled reports (see screenshot)
Image

In addition, setting a start date should also be possible for one-time reports (see design).
Image
@RomijnHumanoids RomijnHumanoids converted this from a draft issue Oct 29, 2024
@jpbruinsslot jpbruinsslot mentioned this issue Nov 13, 2024
Merged
9 tasks
@madelondohmen madelondohmen self-assigned this Nov 14, 2024
@madelondohmen madelondohmen moved this from Backlog / To do to In Progress in KAT Nov 14, 2024
@Rieven Rieven linked a pull request Nov 18, 2024 that will close this issue
Open
9 tasks
@Rieven
Copy link
Contributor

Rieven commented Nov 18, 2024

After meeting with the design team, we have discussed the limitations of using non-native html elements and seperated the time widget from the date widget:

We are going to implement:
image

@Rieven Rieven moved this from In Progress to Review in KAT Nov 18, 2024
@Rieven Rieven self-assigned this Nov 18, 2024
@madelondohmen madelondohmen removed their assignment Nov 19, 2024
@RomijnHumanoids
Copy link
Author

RomijnHumanoids commented Nov 26, 2024 •
edited
Loading

Updated design with elements of previous design (after discussions, there was more possible in terms of implementation)

Updated link: https://www.figma.com/design/5lYYfE1SqJ9qofjyvwdvtw/%E2%9C%85-KAT-%7C-Approved-%7C-Reports-Flow?node-id=2930-7628&t=Kl7gBHpaL2pPqaVR-1

Screenshots:
Screenshot 2024-11-26 at 16 20 41

Screenshot 2024-11-26 at 16 19 48

@madelondohmen madelondohmen moved this from Review to In Progress in KAT Nov 28, 2024
@madelondohmen madelondohmen moved this from In Progress to Review in KAT Dec 12, 2024
@stephanie0x00 stephanie0x00 added the 😸 Review/QA feedback Review/QA feedback provided label Dec 16, 2024
@ammar92 ammar92 moved this from Review to QA review / functional testing in KAT Dec 17, 2024
@madelondohmen madelondohmen assigned ammar92 and unassigned Rieven Dec 17, 2024
@ammar92 ammar92 assigned stephanie0x00 and unassigned ammar92 Dec 18, 2024
@madelondohmen madelondohmen moved this from QA review / functional testing to In Progress in KAT Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Rieven Rieven

Labels
approved for implementation (design done) frontend 😸 Review/QA feedback Review/QA feedback provided
Projects
KAT
Status: In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Changes to schedule all reports, even for once minvws/nl-kat-coordination
5 participants
@ammar92 @stephanie0x00 @Rieven @madelondohmen @RomijnHumanoids