Add cross-platform notifications and setup guide updates#13
Add cross-platform notifications and setup guide updates#13AAYUSH-SPIDEY-SHARMA wants to merge 1 commit into01bps:mainfrom
Conversation
There was a problem hiding this comment.
A lot of changes are made which are outside the scope of the feature.
Also why have you removed sqlite ?
The setup.md file has many necessary steps marked such as for fetching the payload parameters.
Also change in name of variables are cosmetic in nature and should be kept intact.
Kindly resolve these and keep the PR limited to the scope of the feature itself strictly.
And do mention some image or working video of the functionality.
|
Thanks for reviewing this Let me explain the reasoning behind the changes: Notifications SQLite Setup.md Variable name changes |
This PR implements #10 by adding:
Tested on Linux, Windows, and macOS. Falls back to console if notifications unavailable.