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.

Sign up for GitHub

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

Jump to bottom

Notifications fix #1014

Closed

garthvh wants to merge 2 commits into main from notificationsFix

Member

garthvh commented Dec 7, 2024

What changed?

Why did it change?

How is this tested?

Screenshots/Videos (when applicable)

Checklist

My code adheres to the project's coding and style guidelines.
I have conducted a self-review of my code.
I have commented my code, particularly in complex areas.
I have made corresponding changes to the documentation.
I have tested the change to ensure that it works as intended.

RCGV1 added 2 commits

November 30, 2024 19:09


          UNK node notifications and immediate notification scheduling

ad43fc0


          Merge pull request #1010 from RCGV1/main

473edce

Fixes #1007

Member Author

garthvh commented Dec 10, 2024

After thinking about this a bit I think the better approach is to make a user if they don't exist.

garthvh closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet