Skip to content

hyper-table-v2/filtering-renderers/date: allow moving option key to be extended#216

Merged
phndiaye merged 4 commits intomasterfrom
pn/vel-4194
Dec 31, 2024
Merged

hyper-table-v2/filtering-renderers/date: allow moving option key to be extended#216
phndiaye merged 4 commits intomasterfrom
pn/vel-4194

Conversation

@phndiaye
Copy link
Member

What does this PR do?

hyper-table-v2/filtering-renderers/date: allow moving option key to be extended in case the backend the API expects a different value.

What are the observable changes?

🧑‍💻 Developer Heads Up

⚡ Since we are using Ember Octane now:

  • Feel free to migrate existing components to Glimmer Components.
  • Write new ones exclusively in it.

Useful Resource : Ember Octane vs Classic Cheat Sheet

Good PR checklist

  • Title makes sense
  • Is against the correct branch
  • Only addresses one issue
  • Properly assigned
  • Added/updated tests
  • Added/updated documentation
  • Migrated touched components to Glimmer Components
  • Properly labeled

Additional Notes

@phndiaye phndiaye requested a review from a team as a code owner November 15, 2024 09:03
@phndiaye phndiaye requested review from Xegushu, aprentout and nathalieArnoux and removed request for a team November 15, 2024 09:03
@linear
Copy link

linear bot commented Nov 15, 2024

Copy link
Contributor

@JulienVannier66 JulienVannier66 left a comment

Choose a reason for hiding this comment

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

Otherwise LGTM

phndiaye and others added 3 commits December 27, 2024 11:01
* hypertable v2: add support for empty_state_message

* add more test scenarios for rows and fix tests

* hyper-table-v2/filtering-renderers/numeric: add multiplier option, allows for extension in money-type columns
@phndiaye phndiaye merged commit ed9f3d5 into master Dec 31, 2024
1 check passed
@phndiaye phndiaye deleted the pn/vel-4194 branch December 31, 2024 12:12
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.

4 participants

Comments