Laravel-Backpack / theme-tabler

UI for Backpack v6 that uses Tabler and Bootstrap v5.
MIT License
21 stars 12 forks source link

[Bug] Remove url helper from config #58

Closed pxpm closed 1 year ago

pxpm commented 1 year ago

Reported in https://github.com/Laravel-Backpack/theme-tabler/issues/57

TLDR: we cant use url() helper in the config files as it will error and break the application.

I moved the configuration to the ServiceProvider, so instead of providing urls developer now should provide only the url segments and we generate the urls.

I still don't fully like this approach. We can't use the overlap effect in urls with dynamic segments like user/123/profile.

To support that we may need to give it a little bit more thinking on how to match the urls, for now this should fix the error and keep the same behavior.