Skip to content

Conversation

lrljoe
Copy link
Collaborator

@lrljoe lrljoe commented Apr 22, 2025

Fixing broken/missing typehints

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

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:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your core changes, as applicable?
  • Have you successfully ran tests with your changes locally?

Copy link

codecov bot commented Apr 22, 2025

Codecov Report

Attention: Patch coverage is 90.90909% with 1 line in your changes missing coverage. Please review.

Project coverage is 90.19%. Comparing base (6f25fcc) to head (04dc64f).

Files with missing lines Patch % Lines
...TransferObjects/Filters/StandardFilterPillData.php 0.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff               @@
##             development    #2246   +/-   ##
==============================================
  Coverage          90.19%   90.19%           
+ Complexity          2159     2157    -2     
==============================================
  Files                242      242           
  Lines               4865     4865           
==============================================
  Hits                4388     4388           
  Misses               477      477           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@lrljoe lrljoe merged commit c6dc6d8 into rappasoft:development May 2, 2025
@lrljoe lrljoe mentioned this pull request May 3, 2025
lrljoe added a commit that referenced this pull request May 3, 2025
* Link column fix (#2223)

* Add handling for From field for "LinkColumn"

* Fix styling

* Update ChangeLog

* docs: add reminder of requirement to use toggle method (#2208)

Co-authored-by: Joe <[email protected]>

* Update IsNumericFilter.php (#2230)

* Link column fix (#2223) (#2224)

* Link column fix (#2223)

* Add handling for From field for "LinkColumn"

* Update ChangeLog

* Update IsNumericFilter.php

Fixes issues 2229

---------

Co-authored-by: Joe <[email protected]>

* Fix styling

* Boolean filter for bootstrap (#2244)

* Add BooleanFilter option for Bootstrap

* Add BS4 Support

* Update docs

* Fixes for livewire component column (#2245)

* Initial Commit

* Fix styling

* Initial Commit

* Fix styling

* Adjust test

* Fix styling

* Use test-livewire-column-component

* Additional Tests for LivewireComponentColumn

* Fix styling

* Add getContents Test

* TypeHint/PHPDoc Tidying (#2246)

* Fix broken typehints

* Minor tweaks to Footer/SecondaryHeader to ensure presence of callback prior to executing

* Fix styling

* PHPDoc additions

* Fix styling

* Update ChangeLog for TypeHintFixes

* Fix styling

* FilterPillData and StandardFilterPillData phpdocs

* Fix styling

* Add Wrapper Options to ArrayColumn (#2255)

* Add Wrapper Options to FlexCol

* Fix styling

* Moving getContents Back

* Fix styling

* Add Docs

* Add Wrapper Manual Tests

* Fix styling

* Update to test separator

* Fix styling

* Improved pagination UX (#2251)

* Link column fix (#2223) (#2224)

* Link column fix (#2223)

* Add handling for From field for "LinkColumn"

* Update ChangeLog

* Improved pagination UX

---------

Co-authored-by: Joe <[email protected]>

* Fixed collapsed function bug with tailwind CSS on mobile version (#2228)

* v3.7.2 updates (#2256)

---------

Co-authored-by: da-mask <[email protected]>
Co-authored-by: G4Zz0L1 <[email protected]>
Co-authored-by: Daniel Škopek <[email protected]>
Co-authored-by: CarlosChub27 <[email protected]>
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.

1 participant