nav/Yonexus.Client
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Make notifyBotToken/adminUserId optional (unused by client) #2

Open
hzhang wants to merge 1 commits from notify-provider into main
pull from: notify-provider
merge into: nav:main
Conversation 0 Commits 1 Files Changed 3 +22 -14
hzhang commented 2026-05-19 14:59:18 +00:00
Contributor
Copy Link

The client never sends pairing notifications; these Discord fields were required but unused. Now optional + dropped from required schema. Back-compat preserved. Pairs with nav/Yonexus.Server#notify-provider.

The client never sends pairing notifications; these Discord fields were required but unused. Now optional + dropped from required schema. Back-compat preserved. Pairs with nav/Yonexus.Server#notify-provider.
hzhang added 1 commit 2026-05-22 22:25:21 +00:00
feat: make notifyBotToken/adminUserId optional d0b19cf116 
The client never sends pairing notifications (the server does); these
Discord fields were required but unused. Make them optional + drop from
the config schema's required list. Back-compat: still accepted if set.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
hzhang force-pushed notify-provider from 12a0b8f5e6 to d0b19cf116 2026-05-22 22:25:21 +00:00 Compare
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin notify-provider:notify-provider
git checkout notify-provider
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
zhi nav
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: nav/Yonexus.Client#2
Delete Branch "notify-provider"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?