rappasoft / laravel-livewire-tables

A dynamic table component for Laravel Livewire
https://rappasoft.com/docs/laravel-livewire-tables/v2/introduction
MIT License
1.7k stars 320 forks source link

WireLinkColumn #1659

Open lrljoe opened 4 months ago

lrljoe commented 4 months ago

Docs & Tests In Progress

This PR adds in the WireLinkColumn Column Type

This is not ready for merging, as Docs & Tests still need completing.

WireLinkColumn:

Example

WireLinkColumn::make("Delete With Conf")
  ->title(fn($row) => 'Delete With Conf')
  ->confirmMessage('Test')
  ->action(fn($row) => 'delete("'.$row->id.'")')
  ->attributes(fn($row) => [
      'class' => 'btn btn-danger',
  ]),

All Submissions:

New Feature Submissions:

  1. [ ] Does your submission pass tests and did you add any new tests needed for your feature?
  2. [ ] Did you update all templates (if applicable)?
  3. [ ] Did you add the relevant documentation (if applicable)?
  4. [ ] Did you test locally to make sure your feature works as intended?

Changes to Core Features:

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 85.18519% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 88.11%. Comparing base (bed2130) to head (0868d51). Report is 17 commits behind head on develop.

Files Patch % Lines
src/Views/Traits/Core/HasConfirmation.php 42.85% 4 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #1659 +/- ## ============================================= - Coverage 88.13% 88.11% -0.03% - Complexity 1310 1321 +11 ============================================= Files 123 126 +3 Lines 3102 3129 +27 ============================================= + Hits 2734 2757 +23 - Misses 368 372 +4 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

stale[bot] commented 3 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] commented 2 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

lrljoe commented 3 weeks ago

For tracking purposes only - will be merged properly in next 2 weeks