search

Azure / azure-sdk-tools

Tools repository leveraged by the Azure SDK team.
MIT License
111 stars 176 forks source link

Add 'On Maintenance' view in the different apps #3938

Open JonathanCrd opened 2 years ago

JonathanCrd commented 2 years ago

There has been many errors recently that breaks the process of scheduling a meeting. We should have a way to prevent users to use the tool if there is a major issue causing problems, like a On Maintenance message.

We should also let the admins to change the value of whether the app is available or not trough the admin app.

maririos commented 2 years ago

what should users do while the app is on maintenance

JonathanCrd commented 2 years ago

what should users do while the app is on maintenance

They just should wait. The app must display a message like:

Hello, the app is currently on maintenance. Please come back later or contact the following people for questions: For Informational meetings ... email For SDK Review meetings... email For API Review meetings... email

I'm changing the priority on this because a work around we can do right now is to delete all the available time slots from the app, so no user will be able to schedule something and run into issues.

maririos commented 2 years ago

Do you know if by deleting the meeting slots our Dataverse becomus corrupt? i.e.

  1. if someone scheduled a meeting for next week
  2. We remove time slots
  3. We add time slots again
JonathanCrd commented 2 years ago

What happens to that meeting?

Nothing should happen to that meeting. The Event record has a date of date-time type. The time availability just has the week day, hour and minute. What I mean is that there is not a direct relationship between those two tables.

Will the app know that the slot is taken?

Yes, if the calendar detects that there is an event for a certain date and time, it won't show the button to select that time slot. (At least that is the expected behavior, if this is not what happens, then there is a bug)

maririos commented 2 years ago

@JonathanCrd coudl you include the design for this work here?