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

Update DownloadController.php #962

Merged
merged 3 commits into from
Dec 18, 2020
Merged

Update DownloadController.php #962

merged 3 commits into from
Dec 18, 2020

Conversation

FatihKoz
Copy link
Contributor

Changed the group name according to its category , aircrafts will show icao type and registration , airports will be displayed with their icao codes names and countries. Rest will be showed with their names (exp airlines)

This will result a much better look on the downloads page for end users while looking for a specific type's files (like a repaint for a B738).

Changed the group name according to its category , aircrafts will show icao type and registration , airports will be displayed with their icao codes names and countries. Rest will be showed with their names (exp airlines)

This will result a much better look on the downloads page for end users while looking for a specific type's files (like a repaint for a B738).
@FatihKoz
Copy link
Contributor Author

I am no longer trying to fix ridiculous style errors styleci finds @nabeelio , sorry for that ...

1a
1b

It is fine on my end and works fine ... Just downloaded vscode , used it for changes , even tried sending a pr with github desktop it always finds silly style errors while the code is clean and tidy.

So if you want to reject, it is ok for me will never spend my time for it again.

@nabeelio nabeelio added this to the 7.0.0 milestone Dec 18, 2020
@nabeelio
Copy link
Owner

Fixed it for you, will merge once it passes

@FatihKoz
Copy link
Contributor Author

Thanks for your time

@nabeelio nabeelio merged commit e4972ea into nabeelio:dev Dec 18, 2020
@FatihKoz FatihKoz deleted the patch-5 branch December 21, 2020 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants