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: [autogrid] ignore default object json renderer if columnOption path is set #2361

Conversation

rbrki07
Copy link
Contributor

@rbrki07 rbrki07 commented Apr 25, 2024

Description

This should fix that AutoGrid uses the build-in JSON renderer for a column of type object, although a custom path is set for this column. Instead, it should use the default string renderer for the property target with path.

Fixes #2322

Type of change

  • Bugfix
  • Feature

Checklist

  • I have read the contribution guide: https://vaadin.com/docs/latest/guide/contributing/overview/
  • I have added a description following the guideline.
  • The issue is created in the corresponding repository and I have referenced it.
  • I have added tests to ensure my change is effective and works as intended.
  • New and existing tests are passing locally with my change.
  • I have performed self-review and corrected misspellings.

@taefi taefi changed the title fix: autogrid uses default renderer for column type object, if custom… fix: [autogrid] ignore default renderer for column type object if custom path is set Apr 29, 2024
Copy link

codecov bot commented Apr 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.13%. Comparing base (de06c3d) to head (f9fcab3).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2361      +/-   ##
==========================================
- Coverage   95.17%   95.13%   -0.05%     
==========================================
  Files          66       66              
  Lines        4499     4500       +1     
  Branches      644      645       +1     
==========================================
- Hits         4282     4281       -1     
- Misses        177      179       +2     
  Partials       40       40              
Flag Coverage Δ
unittests 95.13% <100.00%> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@taefi taefi self-assigned this Apr 29, 2024
taefi

This comment was marked as outdated.

@taefi taefi requested a review from Lodin April 29, 2024 10:03
@taefi taefi changed the title fix: [autogrid] ignore default renderer for column type object if custom path is set fix: [autogrid] ignore default object json renderer if columnOption's path is set Apr 29, 2024
@taefi taefi changed the title fix: [autogrid] ignore default object json renderer if columnOption's path is set fix: [autogrid] ignore default object json renderer if columnOption path is set Apr 29, 2024
Copy link

sonarcloud bot commented May 2, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@rbrki07 rbrki07 requested a review from taefi May 2, 2024 04:55
Copy link
Contributor

@taefi taefi left a comment

Choose a reason for hiding this comment

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

Nice work @rbrki07! LGTM!

@taefi taefi merged commit eb8d66a into vaadin:main May 2, 2024
16 of 17 checks passed
@rbrki07 rbrki07 deleted the fix/autogrid-uses-default-renderer-for-column-type-object-if-custom-path-is-set branch May 2, 2024 11:44
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Hilla 24.4.0.beta2 and is also targeting the upcoming stable 24.4.0 version.

@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Hilla 24.5.0.alpha1 and is also targeting the upcoming stable 24.5.0 version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AutoGrid uses JSON renderer for column of type object, although a custom path is set.
3 participants