Skip to content

Conversation

@nickvergessen
Copy link
Member

…yone

☑️ Resolves

grafik

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not possible
  • 📘 API documentation in docs/ has been updated or is not required
  • 🔖 Capability is added or not needed

Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Very nice! Hopefully less notification spam on one side, and less embarrassment on the other :)

@nickvergessen nickvergessen force-pushed the bugfix/noid/outline-the-all-count branch 2 times, most recently from 672046e to e517145 Compare July 23, 2024 11:03
Copy link
Contributor

@Antreesy Antreesy left a comment

Choose a reason for hiding this comment

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

LGTM

@nickvergessen
Copy link
Member Author

nickvergessen commented Jul 23, 2024

@nickvergessen nickvergessen force-pushed the bugfix/noid/outline-the-all-count branch from e517145 to e75a811 Compare July 23, 2024 13:32
@nickvergessen nickvergessen enabled auto-merge July 23, 2024 13:32
@nickvergessen nickvergessen merged commit 78f4009 into main Jul 23, 2024
@nickvergessen nickvergessen deleted the bugfix/noid/outline-the-all-count branch July 23, 2024 13:48
@danxuliu danxuliu mentioned this pull request Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants