daveroverts / bmac Goto Github PK

There are some cases where bookings are imported, however, due to an mistake in the import file, some / all bookings have wrong values.

For a while, removing all bookings was done by either re-creating the event, manually removing bookings 1 by 1, or running a query DELETE FROM bookings WHERE event_id = ?. Foreign keys are already in place to make sure data between bookings and flights are deleted as needed.

It would save time if there is a button called Delete all bookings that removes all bookings. Because import is usually done after creating event, I'm not planning on adding a extra check if a flight is already booked or not.

When you're on the My settings page, the active class is active on the 2 items within the dropdown

For #193 I need LAT LON data to show airports. Right now, they're not saved at all. The import file that's already in use does have such data.

When importing over 400 flights, chances are that a memory limit might be reached...

Since the introduction of Livewire for the Bookings Overview, the Flights available numbers don't update the moment when you filter, those just stick with values based on Show All.

Instead of using the airport import script (that loads in airports we don't need), it would be nice to have them auto-added to the database if it's missing while running a booking import.

Low priority, #432 adds the ability to also search through airports

Hello!
I've got issue with Book button.
If someone booked the slot already - I still have "book now" button.

And if you click on it - it tells that the slot is booked by someone.

The next issue is that if I book the slot - I can't drop it.

It tells me "you already have a booking". Only way what I can do - fully remove the slot to release it. (via admin)

Seems that Vatsim allows e-mails to be used on multiple accounts...

Request: Add the following fields to app/Http/Resources/EventResource.php:

• url: URL of the event. Example: 'https://bmac.daveroverts.nl/random-test-event/bookings'.
• total_bookings_count: [int] Total bookings that are in the system for the event. Example: 30.
• available_bookings_count: [int] Bookings that are still available to be booked. Example: 28.

The data we need is already shown in the bookings table.

Note 2 self: #375 has edited that file a bit. Maybe merge that in first before beginning with this issue.

When downloading the slot template csv, one column is named EOBT, while when uploading the site is expecting CTOT.

A nice follow-up on #119 would be to make the e-mail editable (within limits).

Initially, I'm thinking of this order:

• Salutation
• Content
• Booking details

What also would be possible is to use something like [booking_details] within tinymce to show all details there instead

It seems that there is a syntax error when on the bookings table (i.e. /bookings). Get this error: syntax error, unexpected 'Event' (T_STRING), expecting function (T_FUNCTION) or const (T_CONST) (View: /var/www/booking/resources/views/booking/overview.blade.php)

Using dev branch (latest). Master branch has no difference.

The import script is fun and all, but maybe it's a good idea to implement a script that can delete airports that haven't been used.

The following conditions should be applied:

• Not coupled to a Event
• Not coupled to a Booking
• Not coupled to a AirportLink

Right now, the usage of accessors and raw values are mixed, this needs to be corrected in the whole application.

• Accessors: Should be used within views to output something
• Raw values: Must be used in forms

Right now, finalInformation.blade.php is static, and focussed on one specific event. This could use some improvements.

Initially, I think It's a good idea if a more generic template is used. There already is one available at finalInformation_multiflights.blade.php

Do take note of empty variables.

Times currently use the default <input type="time">, good if your browser language is something like Dutch, but not if it's in something like English, where the 12h format is shown

If there are no events, the divider should be hidden.

Hi,

I have set up the project as described and it works well except for the fact that the description field on the events page does not show up. It seems to be because it has visibility is hidden and the TinyMCE is not activating correctly. I have attached screenshots of a couple of things I noticed that seem to be related. I'm not all that familiar with Laravel otherwise I would have fixed it myself.

Screenshots:
Add/edit event page

DOM of descripion field

Last tag in head

Any assistance would ve appreciated. Thanks

protonemedia/laravel-form-components has been deprecated. This causes issues in the future as it's locked to Laravel 10 and Livewire 2.

If I ever start with #432, this needs to be done first.

I know one environment that doesn't send out E-mails out at all. However, some flash messages clearly show a e-mail was sent.

The following lines show this:

Now the question is: Do I want to remove that bit of message al together, or only show it if env actually sends out e-mails?

Now that I've added Laravel Livewire to bookings.overview, it would be nice to also add Laravel Echo for that page, so that you can see each booking change depending on what's happening.

In my opinion, this whole feature should be optional. Events will always broadcast, but if you don't setup broadcasting, nothing happens on the client side.

If there's no upcoming event, the dropdown should not be visible.

Related with #406. Bootstrap 5 dropped jQuery as required dependency.

Note to self: Do this after #432 to reduce potential unneeded work

I once implemented a option to hide events from homepage, same should also apply for the navbar

Might've missed a upgrade guide or something, as it's somewhat broken now

https://filamentphp.com/docs/2.x/admin/installation

Hello, i make the first Artisan Migrate from last release

this is a problem from my side or migrate side ?

When you login to bmac for the very first time, something happens in the redirect phase, causing you to not login. However, when you press the login button for the second time, it does work.

Right now, some e-mails contain the event name in the subject, some don't. It's better to always include it.

There are cases where a admin wants to create bookings, but don't want to require either departure or arrival airport.

The easy fix is to create a dummy airport 'ZZZZ', but it would be better if admin doesn't have to fill anything in there

For Fly-Ins, it would be nice to show a map where you see all flights.

In #122 , I replaced the the datepickers (jQueryUI) with flatpickr + Alpine.js. It would be nice to get rid of jQuery within the rest of the views.

Note to self: Do this after #432 to reduce potential unneeded work

Not everybody will use e-mails, making most notifications useless. I think it's best to add notifications in the front-end.

The table already exists (and I checked, no difference from 5.6 to 7), so only 2 things need to be done:

• Notifications not triggered by user, add database to via
• Create something in the navbar so users can actually read the notifications