- Issue created by @jurgenhaas
- πΊπΈUnited States josh.fabean
I agree, based on the API call the "one-click" booking is doing the success message set on the Calendar should already be there available, and it just needs to be printed onto the page. The rest of those features also make sense and should get implemented.
- π©πͺGermany jurgenhaas Gottmadingen
@josh.fabean I started looking into this and saw that the one-click booking is not using the Drupal ajax API and I'm uncertain if that could be easily changed without breaking other things. If we changed that, the API controller would respond with an AjaxResponse instead of a JsonResponse and that would allow all sorts of nice things like e.g. adding a MessageCommand which would then be automatically handled without us having to do all that manually.
Is the API used by other clients too or is it just for the one-click booking and could be updated?
- πΊπΈUnited States josh.fabean
That API call is currently only used inside the module for the one click booking, but it's existed for a while and how to use it is on the README so how many people are using it would be impossible to know. So I think that leaves the options of, creating a new API endpoint to not break this one, or update this one and move it to a version of 2.3.x and list it as a potential breaking change with upgrade instructions.
- π©πͺGermany jurgenhaas Gottmadingen
OK, I'll go for a separate endpoint then.
- @jurgenhaas opened merge request.
- π©πͺGermany jurgenhaas Gottmadingen
The MR now contains the re-implementation of the one-click-booking with Drupal's ajax and it also displays the message(s) collected by Drupal during that request. Adding the same for cancellation will then be addressed in the other issue.
- Status changed to Needs review
over 1 year ago 10:03am 27 April 2023 - π©πͺGermany jurgenhaas Gottmadingen
Sorry @josh.fabean for my late response, I've now fixed that last issue.
- π©πͺGermany jurgenhaas Gottmadingen
Just committed https://git.drupalcode.org/project/bookable_calendar/-/merge_requests/28... to address the cancellation emails from β¨ Improve booking cancellation Fixed
Now working on the on-click-cancellation feature.
- πΊπΈUnited States josh.fabean
Thanks, I'll get this tested and merged in this week.
- π©πͺGermany jurgenhaas Gottmadingen
OK, I've now committed https://git.drupalcode.org/project/bookable_calendar/-/merge_requests/28... which does the one-click-cancel. The JS code is not complete yet, but it's functional and can be tested.
- π©πͺGermany jurgenhaas Gottmadingen
Yeah, the last commit at https://git.drupalcode.org/project/bookable_calendar/-/merge_requests/28... resolved the final remaining issues from my POV. It's ready to go, at least for your review of course ;-)
Really looking forward to your response.
-
josh.fabean β
committed 21409404 on 2.2.x authored by
jurgenhaas β
Issue #3349330: Display booking success message also when one-click...
-
josh.fabean β
committed 21409404 on 2.2.x authored by
jurgenhaas β
- Status changed to Fixed
over 1 year ago 3:32am 9 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.