Skip to content

Conversation

theHocineSaad
Copy link
Contributor

This PR introduces an admin panel built with Filament.

I'm aware of #1179, but I think the contributor doesn't have time to continue it, and they haven't made much progress. Also, Filament 3 (not 4) was used, so I believe this PR is justified. If this is a wrong move, I'm truly sorry, @ijpatricio.

@driesvints
Note: The old admin panel still exists at its original route /admin, while the new one is located at /admin-filament.

image image image image

@ijpatricio
Copy link

Hey @theHocineSaad , not al all!! I'm really happy someone is making the push!!

I've been not having time, it happens to us all, I didn't want to give up because I eventually will be more free, still this year, but in the end, what matters is that it gets DONE, so THANK YOOOOU! ❤️

@driesvints
Copy link
Member

Thanks a lot for this @theHocineSaad! you can rip the old admin panel out and move the filament one to /admin instead.

Thanks also for kickstarting this @ijpatricio !

@theHocineSaad
Copy link
Contributor Author

Hey @theHocineSaad , not al all!! I'm really happy someone is making the push!!

I've been not having time, it happens to us all, I didn't want to give up because I eventually will be more free, still this year, but in the end, what matters is that it gets DONE, so THANK YOOOOU! ❤️

You're welcome .. Thanks for your understanding .. Glad to help

@theHocineSaad
Copy link
Contributor Author

Thanks a lot for this @theHocineSaad! you can rip the old admin panel out and move the filament one to /admin instead.

Thanks also for kickstarting this @ijpatricio !

You're welcome .. done => 6071d21 .. I just need to update some tests in AdminTest.php

@theHocineSaad theHocineSaad marked this pull request as draft September 3, 2025 18:31
@theHocineSaad theHocineSaad marked this pull request as ready for review September 3, 2025 19:45
@theHocineSaad theHocineSaad marked this pull request as draft September 5, 2025 16:20
@ijpatricio
Copy link

Thanks a lot for this @theHocineSaad! you can rip the old admin panel out and move the filament one to /admin instead.

Thanks also for kickstarting this @ijpatricio !

My absolute pleasure, @driesvints

I'm just sorry I couldn't find more time, but this is the magic of open source, @theHocineSaad came and joined!!

@theHocineSaad theHocineSaad marked this pull request as ready for review September 6, 2025 09:44
@theHocineSaad
Copy link
Contributor Author

@driesvints Just following up on this PR. Please let me know if there’s any feedback.

@driesvints
Copy link
Member

Thanks @theHocineSaad. Been a bit under snowed with other work and PR's but planning on getting this one in soonish.

@theHocineSaad
Copy link
Contributor Author

Thanks @theHocineSaad. Been a bit under snowed with other work and PR's but planning on getting this one in soonish.

Ok great

@driesvints driesvints merged commit 5b85454 into laravelio:main Sep 19, 2025
1 check passed
@driesvints
Copy link
Member

Finally got this merged. I reverted some queueing you added to a few jobs, not sure if that was necessary? These actions should execute immediately so you see the effect on the subsequent page reload.

Thanks a bunch you to @ijpatricio & @theHocineSaad! I really appreciate the help with this one 😄

@driesvints
Copy link
Member

Ah one thing I notice now in production: a link to get back to the main website in the nav somewhere would be great.

@theHocineSaad
Copy link
Contributor Author

Finally got this merged. I reverted some queueing you added to a few jobs, not sure if that was necessary? These actions should execute immediately so you see the effect on the subsequent page reload.

Thanks a bunch you to @ijpatricio & @theHocineSaad! I really appreciate the help with this one 😄

You're welcome.

Without the Queueable trait, you will get this error:

image

And I used dispatchSync so the job will not be queued and will run immediately.

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

Successfully merging this pull request may close these issues.

3 participants