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: Make new booking modal to be client side #1610

Closed
4 tasks
DonKoko opened this issue Jan 29, 2025 · 0 comments · Fixed by #1621
Closed
4 tasks

Feature: Make new booking modal to be client side #1610

DonKoko opened this issue Jan 29, 2025 · 0 comments · Fixed by #1621
Assignees
Labels
enhancement New feature or request

Comments

@DonKoko
Copy link
Contributor

DonKoko commented Jan 29, 2025

Okey so we want to make the modal for new booking client side, however we still want to keep the route. Let me sum up the tasks:

  • Create a client side modal for new booking
  • When clicking the "New booking" button on bookings page, open the client side modal
  • Add a "New booking" button on calendar page, that will also open the client side modal
  • Keep the route bookings/new so if someone navigates to it in the browser it still works

We might have to create a bunch of functions that handle both the loader and the action for new booking as we would need to have 2 routes: the current booking.new and a resource route that doesn't return HTML but just JSON.

Let me know if there are any questions.

@DonKoko DonKoko converted this from a draft issue Jan 29, 2025
@DonKoko DonKoko added the enhancement New feature or request label Jan 29, 2025
@rockingrohit9639 rockingrohit9639 moved this from 🗓️ Planned to 🏗 In progress in 🗺️ Shelf Roadmap Feb 3, 2025
@rockingrohit9639 rockingrohit9639 moved this from 🏗 In progress to 👀 In review in 🗺️ Shelf Roadmap Feb 5, 2025
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in 🗺️ Shelf Roadmap Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants