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

[1.x] Remove Hard Coded Livewire Routes #226

Merged
merged 3 commits into from
Sep 18, 2020
Merged

[1.x] Remove Hard Coded Livewire Routes #226

merged 3 commits into from
Sep 18, 2020

Conversation

deanlj
Copy link

@deanlj deanlj commented Sep 18, 2020

Only for Livewire, I don't use Vue so cannot test if these changes would work in the vue component.

Ensures the correct endpoints are rendered in the html should the underlying route definition change

Only for Livewire, I don't use Vue so cannot test if these changes would work in the vue component
{{ __('Profile') }}
</x-jet-responsive-nav-link>

@if (Laravel\Jetstream\Jetstream::hasApiFeatures())
<x-jet-responsive-nav-link href="/user/api-tokens" :active="request()->routeIs('api-tokens.index')">
<x-jet-responsive-nav-link href="{{ route('/user/api-tokens') }}" :active="request()->routeIs('api-tokens.index')">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't a route name.

@taylorotwell
Copy link
Member

Has this been tested? One of the route names is not a route name at all.

@deanlj
Copy link
Author

deanlj commented Sep 18, 2020

sorry, fat fingers pasting by accident before commit

@taylorotwell taylorotwell merged commit 05c4ef6 into laravel:1.x Sep 18, 2020
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.

None yet

2 participants