Skip to content

Added locked state on list cell renderer#220

Merged
OwenCoogan merged 6 commits intomasterfrom
oc/DRA-1722
Jan 9, 2025
Merged

Added locked state on list cell renderer#220
OwenCoogan merged 6 commits intomasterfrom
oc/DRA-1722

Conversation

@OwenCoogan
Copy link
Contributor

What does this PR do?

Related to : #

What are the observable changes?

Capture d’écran 2024-12-30 à 17 44 51
Capture d’écran 2024-12-30 à 17 43 58
Capture d’écran 2024-12-30 à 17 14 30
Capture d’écran 2024-12-30 à 17 14 20
Capture d’écran 2024-12-30 à 17 13 45

🧑‍💻 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

@OwenCoogan OwenCoogan self-assigned this Dec 30, 2024
@OwenCoogan OwenCoogan requested review from a team and phndiaye as code owners December 30, 2024 16:50
@OwenCoogan OwenCoogan requested review from JulienVannier66 and Xegushu and removed request for a team December 30, 2024 16:50
@linear
Copy link

linear bot commented Dec 30, 2024

Copy link
Member

Choose a reason for hiding this comment

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

can you make this a computed instead ?

Comment on lines 89 to 95
Copy link
Member

Choose a reason for hiding this comment

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

not super fan of having this in this project tbh. it's highly bound to upfluence context which this project isn't. we can discuss it live if you want

@OwenCoogan OwenCoogan merged commit bab25a3 into master Jan 9, 2025
3 checks passed
@OwenCoogan OwenCoogan deleted the oc/DRA-1722 branch January 9, 2025 16:59
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.

5 participants

Comments