Skip to content

Add events when apply range filters#253

Merged
JulienVannier66 merged 1 commit intojv/affiliate-improvements-fixesfrom
jv/vel-6162/sales-value-filter-jeton-sometimes-disappears-when-closing-the-filter
Sep 4, 2025
Merged

Add events when apply range filters#253
JulienVannier66 merged 1 commit intojv/affiliate-improvements-fixesfrom
jv/vel-6162/sales-value-filter-jeton-sometimes-disappears-when-closing-the-filter

Conversation

@JulienVannier66
Copy link
Contributor

What does this PR do?

Add events when apply range filters

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

@JulienVannier66 JulienVannier66 self-assigned this Sep 3, 2025
@JulienVannier66 JulienVannier66 requested review from a team and phndiaye as code owners September 3, 2025 14:14
@JulienVannier66 JulienVannier66 requested review from Elodie-DeMatteis-Upf and aprentout and removed request for a team September 3, 2025 14:14
@linear
Copy link

linear bot commented Sep 3, 2025

@JulienVannier66 JulienVannier66 force-pushed the jv/vel-6162/sales-value-filter-jeton-sometimes-disappears-when-closing-the-filter branch from 6c62604 to 33ed397 Compare September 4, 2025 07:46
@JulienVannier66 JulienVannier66 merged commit c1b5259 into jv/affiliate-improvements-fixes Sep 4, 2025
3 checks passed
@JulienVannier66 JulienVannier66 deleted the jv/vel-6162/sales-value-filter-jeton-sometimes-disappears-when-closing-the-filter branch September 4, 2025 08:10
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