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

fix: respect the model label defined in the resource class when using ExportAction 3.x #15237

Conversation

borjajimnz
Copy link
Contributor

Description

As described in #13643, I'm not sure if this approach is the best, but it does fix the bug.

The thing is, the only way I found to get the model label is through the Livewire / table component. Is there another way to do this?

Functional changes

  • Code style has been fixed by running the composer cs command.
  • Changes have been tested to not break existing functionality.
  • Documentation is up-to-date.

@borjajimnz borjajimnz changed the title BUG FIX - Check if table component has the model label fix: Check if table component has the model label Jan 4, 2025
@borjajimnz borjajimnz changed the title fix: Check if table component has the model label fix: respect the model label defined in the resource class when using ExportAction 3.x Jan 4, 2025
@danharrin
Copy link
Member

I think this has been addressed in v4 as part of an architecture rethink, but this fix isn't correct as I don't think you can pass a custom model label just for this action anymore

@danharrin danharrin closed this Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants