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

Feature: Add wizard to fetch list times from calendar #58

Merged
merged 2 commits into from
Aug 29, 2024

Conversation

gingemarsson
Copy link
Contributor

This is a bit of an experiment which I am still not sure if I like, but I would like some input from others.

We have a few bookings each year where each equipment list actually represent different events, primarily during the receptions but also for spexes with multiple rehearsals etc. If you have a lot of equipment lists it can be quite a hassle to input the wildly different times for each equipment list manually, which often is made extra frustrating since that data is already available in the calendar but cannot be imported in any way.

We have already decided that we do not want to change the primary connection between a booking in stage and an calendar event, but I wanted to test a looser connection in the form of a one-off fetch-from-calendar-action, similar to the calendar integration for equipment in/out times. From a UX perspective I built this in the style of the time estimate wizard, which has a very similar behavior for the user: an optional way to fill in the form which may be very useful in some cases but also cannot be used in all cases.

My opinion so far is, I think, that I like the behavior, but I do not like how cluttered the dialog becomes. It is hard to find the primary controls. If there are any suggestions for how to model the UX of the dialog in a better way, I would appreciate them. For now, I went with the "when in doubt, make it similar to other things" path and made it as identical to the time estimate wizard as possible.


image

image

image

@m4reko
Copy link
Contributor

m4reko commented Aug 6, 2024

I don't remember discussing adding connection to multiple calendar events. What was the reason for not making the connection booking <-> calendar one to many? When we have this nice wizard, why not just migrate the connection to EquipmentList level? Should be a straight forward migration since we can just add the field to all lists and fill in from booking field.

Maybe the use case is to narrow for this kind of effort but I think it feels pretty close when you already started with this.

@gingemarsson gingemarsson marked this pull request as ready for review August 29, 2024 18:16
@Herkarl Herkarl merged commit 8c9873b into main Aug 29, 2024
3 checks passed
@Herkarl Herkarl deleted the feature/fetch-calendar-datetimes-to-equipment-list branch August 29, 2024 18:17
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