Skip to content

Conversation

@olxmpe
Copy link
Contributor

@olxmpe olxmpe commented Sep 19, 2025

What does this PR do?

Related to : #

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

@olxmpe olxmpe self-assigned this Sep 19, 2025
@linear
Copy link

linear bot commented Sep 19, 2025

@olxmpe olxmpe changed the title Document hypertable V2 - WIP Document hypertable V2 Nov 26, 2025
@olxmpe olxmpe marked this pull request as ready for review November 26, 2025 08:57
@olxmpe olxmpe requested a review from a team as a code owner November 26, 2025 08:57
@olxmpe olxmpe requested review from OwenCoogan, aprentout and nathalieArnoux and removed request for a team November 26, 2025 08:57
Copy link
Member

@phndiaye phndiaye left a comment

Choose a reason for hiding this comment

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

Niiice!

Image

Copy link
Member

@Miexil Miexil left a comment

Choose a reason for hiding this comment

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

Nicely done πŸ‘

Copy link
Contributor

@Elodie-DeMatteis-Upf Elodie-DeMatteis-Upf left a comment

Choose a reason for hiding this comment

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

Some nit comments and questions, great job!

return this._lookupComponent(columnDef, 'header');
}

private _lookupComponent(
Copy link
Contributor

Choose a reason for hiding this comment

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

I think _ for private functions is a javascript habit no? I guess you could remove it here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's to differentiate this function from the one it calls (super.lookupComponent)


#### Faceted Filtering

Hypertable provides built-in support for **faceted filtering**, allowing users to select multiple values for a column filter from a dynamic list of facets.
Copy link
Contributor

Choose a reason for hiding this comment

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

It may be a dumb question, but what does "faceted" mean in this context?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The normal filters are the ones like with value / without value:

Capture d’écran 2025-11-27 aΜ€ 17 33 10

The 'faceted' ones are the ones that need a list of values:

Capture d’écran 2025-11-27 aΜ€ 17 34 39

Comment on lines +337 to +341
- Fetches facets when the filter UI opens
- Displays a skeleton loader while loading
- Supports search with a 300 ms debounce
- Allows multiple facet selection
- Sorts facets by `count` descending by default (custom sorting function can be provided)
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here, dumb question but what is a "facet"? ^^'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

and facets are the elements in the list πŸ™‚

@olxmpe olxmpe merged commit 173b821 into master Nov 27, 2025
3 checks passed
@olxmpe olxmpe deleted the ol/vel-6121 branch November 27, 2025 16:39
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.

7 participants