-
Notifications
You must be signed in to change notification settings - Fork 135
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
SimBrief Planning System Update #991
Conversation
Replaces outdated old pr. Same improvements apply to both controller and simbrief_form.blade.
resources/views/layouts/default/flights/simbrief_form.blade.php
Outdated
Show resolved
Hide resolved
resources/views/layouts/default/flights/simbrief_form.blade.php
Outdated
Show resolved
Hide resolved
changed equals to strict equals and put in some comments about the loadarray and manualrmk generation.
Changed some formatting (no tabs, only spaces), minor things. Converted |
I do not recall using any @secstohhmm in the simbrief_form ? Anyway, will pull and check both visually and on my test server. |
We have the $wakecat, $equipment and $transponder in the controller, removed them from the blade. Fixed a harmless typo in the IVAO Prefile DEPTIME line
Tested both generating new briefings (long and short hauls with different aircrafts, with different seat caps fare classes etc) ... simbrief_form seems working fine. Also the secstohhmm is working fine in the simbrief_briefing blade. Why the test is failing at acars route part i have no idea. |
Not sure why either, just needed to be restarted. Finally merged, yay! Thank you for all of your work on this |
Yes, finally ... Thank you too ;) Without your assistance this may not be possible at all. I am pleased to have it in the default dev build, hope all users will like it as I did :) |
Replaces outdated old pr.
Same improvements apply to both SimBrief Controller and flights / simbrief_form.blade