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

Added a time slot notification for maintenance mode #267

Merged
merged 1 commit into from
Sep 14, 2023

Conversation

SanttuA
Copy link

@SanttuA SanttuA commented Sep 13, 2023

Time slot error notification for maintenance mode

Changes:

  • added custom error notification for time slot selecting when maintenance mode is on
  • added Redux state handling for maintenance mode reading and writing

Related Trello card


Breakdown:

Maintenance Redux state handling

  1. app/actions/announcementActions.js

    • redux action to set maintenance mode status
  2. app/constants/ActionTypes.js

    • redux action type for maintenance status setting
  3. app/state/reducers/ui/maintenanceReducer.js

    • redux reducer for maintenance status
  4. app/shared/service-announcement/ServiceAnnouncement.js

    • set redux state for maintenance status after fetching announcements

Maintenance error message

  1. app/pages/resource/reservation-calendar/time-slots/TimeSlot.js
    • if maintenance mode is on, display maintenance mode notification before anything else when user tries to select a time slot for a reservation

Changes:
- added custom error notification for timeslot selecting when maintenance mode is on
- added redux state handling for maintenance mode reading and writing
@SanttuA SanttuA requested a review from ezkat September 13, 2023 10:30
@SanttuA SanttuA merged commit bd8c3ec into develop Sep 14, 2023
4 checks passed
@SanttuA SanttuA deleted the feature/custom-timeslot-error-for-maintenance-mode branch January 9, 2024 05:52
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.

2 participants