Skip to content

Conversation

@ShGKme
Copy link
Contributor

@ShGKme ShGKme commented Apr 2, 2025

☑️ Resolves

  • getParent uses "hacky" solution searching for a parent via $parent in a loop
  • It is only used for NcActions to close the menu
  • Instead, explicitly provide an interface for closing the menu
  • Also, use typed injection keys

@susnux, sorry, this test didn't last long 🙇

🏁 Checklist

  • ⛑️ Tests are included or are not applicable
  • 📘 Component documentation has been extended, updated or is not applicable
  • 2️⃣ Backport bugfixes to stable8 for maintained Vue 2 version.

ShGKme added 2 commits April 2, 2025 23:01
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
@ShGKme ShGKme added 3. to review Waiting for reviews feature: actions Related to the actions components refactor ♻️ Pull request that is neither a fix nor a feature labels Apr 2, 2025
@ShGKme ShGKme requested review from Antreesy and susnux April 2, 2025 21:08
@ShGKme ShGKme self-assigned this Apr 2, 2025
Copy link
Contributor

@susnux susnux left a comment

Choose a reason for hiding this comment

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

😍

@ShGKme ShGKme requested a review from skjnldsv April 3, 2025 12:50
Copy link
Contributor

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

Stop it you wizard! 🧙 😁

@skjnldsv skjnldsv merged commit 4858de7 into main Apr 3, 2025
24 checks passed
@skjnldsv skjnldsv deleted the chore/remove-getParent branch April 3, 2025 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews feature: actions Related to the actions components refactor ♻️ Pull request that is neither a fix nor a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants