integreat-app icon indicating copy to clipboard operation
integreat-app copied to clipboard

Add "Only by appointment" to the opening hours of POIs

Open LeandraH opened this issue 1 year ago • 7 comments

Is your feature request related to a problem? Please describe. As described in https://github.com/digitalfabrik/integreat-cms/issues/2398, there are POIs that are only "open" if you have an appointment. The CMS ticket for it is almost done, we should figure out how to handle this in the app.

Describe the solution you'd like As a user of the Integreat App I want to see on which days I need an appointment to go to a POI so that I do not waste my time and have all the relevant information before going.

Describe alternatives you've considered We could ignore it, which seems like a terrible idea since it will soon be possible to add the information in the CMS.

Additional information This draft PR might be a good point to start or take code from: https://github.com/digitalfabrik/integreat-app/pull/2722

LeandraH avatar Sep 21 '23 10:09 LeandraH

@hauf-toni Sorry, here is the issue I meant.

LeandraH avatar Sep 21 '23 10:09 LeandraH

Design proposal can be found 🚩 here

hauf-toni avatar Oct 04 '23 07:10 hauf-toni

Looks good though if I understand the CMS ticket correctly, the "open by appointment"-label applies to specific days and not everything. Sorry, I didn't realize that until now. But I guess we can simply move the text "Termin erforderlich, [...]" to under the days where you need an appointment?

LeandraH avatar Oct 04 '23 08:10 LeandraH

Looks good though if I understand the CMS ticket correctly, the "open by appointment"-label applies to specific days and not everything. Sorry, I didn't realize that until now. But I guess we can simply move the text "Termin erforderlich, [...]" to under the days where you need an appointment?

An alternative design proposal can be found on figma as well

hauf-toni avatar Oct 04 '23 09:10 hauf-toni

Looks good to me, I think we can start working with that :)

LeandraH avatar Oct 16 '23 07:10 LeandraH

Blocked by

  • [x] https://github.com/digitalfabrik/integreat-cms/issues/2398

ztefanie avatar Oct 18 '23 06:10 ztefanie

The corresponding CMS issue is now fixed, the opening hours contain an additional boolean field appointmentOnly now.

timobrembeck avatar Oct 25 '23 17:10 timobrembeck