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

Upgrade to laravel 9 #34

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Conversation

mdprotacio
Copy link
Contributor

upgrade dependencies and laravel

@bkintanar
Copy link
Owner

this is not backward compatible with L8

@mdprotacio
Copy link
Contributor Author

yes it is not. do we want to have it stuck on laravel 8? I'm updating this and would want to run it on octane(swoole).

@mdprotacio
Copy link
Contributor Author

Laravel 10 is scheduled to be released in Feb 2023 and Laravel 9 is already at 9.45.1. It's up to you if you want this to be merged and updated but I'll have my fork updated to 9. :)

@mdprotacio
Copy link
Contributor Author

By the way, I'm checking on the possibility of using the xlsx file instead of CSV and there are some updates also on the codes from PSGC where the codes are now 10 digits and some rows have no codes like the 4 districts of Metro Manila.

@bkintanar
Copy link
Owner

bkintanar commented Dec 24, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants