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

Compatibility with Laravel's Standard Model Directory Structure #4

Open
ShutoYamada opened this issue Oct 31, 2023 · 2 comments
Open
Assignees

Comments

@ShutoYamada
Copy link

Hello,

I am looking into using the authlete-php-laravel library for a project. However, I noticed that in the newer versions of Laravel, it is standard to place model files in the app/Models/ directory, as opposed to the older practice of placing them directly in the app/ directory.

I wanted to check if this library has been updated to accommodate this new standard. If not, are there any plans to update the repository to adhere to this standard in the near future? Adhering to Laravel's latest standards would be beneficial for developers looking to integrate this library into their projects running on the latest version of Laravel.

Thank you for your time and looking forward to your response.

Best regards.

@hefa hefa self-assigned this Oct 31, 2023
@hefa
Copy link
Member

hefa commented Oct 31, 2023

Hi, we have also noticed this issue and at some point we need to update this library for the latest versions of PHP, Laravel, and dependencies. However, the timing of this work is not yet decided.

@ShutoYamada
Copy link
Author

Thank you for your response. I understand the situation and look forward to the library being updated for the latest versions of PHP, Laravel, and dependencies.

I had been troubled by the discrepancy between app/Models/User and app/User dependencies, but I was able to work around this by adding class_alias(App\Models\User::class, App\User::class); in bootstrap/app.php.

Therefore, I plan to proceed using this workaround for the time being.

Thank you once again for your attention to this issue.

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

No branches or pull requests

3 participants