Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(neon_framework): User status DND takes precedence over emoji #1664

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

provokateurin
Copy link
Member

No description provided.

@Leptopoda
Copy link
Member

Shouldn't be that hard to add unit tests or am I mistaken?
If so please do so.

@provokateurin
Copy link
Member Author

Yeah, but it will require adding tests for the whole widget which I didn't have the time for yet.

@provokateurin provokateurin force-pushed the fix/neon_framework/user-status-dnd-over-emoji branch from edd24bf to 626d25b Compare February 28, 2024 21:12
@provokateurin
Copy link
Member Author

@Leptopoda I added tests (and split up the logic so it is easier to test).

@provokateurin provokateurin force-pushed the fix/neon_framework/user-status-dnd-over-emoji branch from 626d25b to a4f83c1 Compare February 28, 2024 21:14
Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin force-pushed the fix/neon_framework/user-status-dnd-over-emoji branch 2 times, most recently from 258cb11 to 8754189 Compare February 28, 2024 21:18
@provokateurin provokateurin merged commit f552ae0 into main Feb 28, 2024
8 checks passed
@provokateurin provokateurin deleted the fix/neon_framework/user-status-dnd-over-emoji branch February 28, 2024 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants