Repository navigation
Conversation
Signed-off-by: Zhu Chenrui <boomzero_zcr@outlook.com>
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThe PR updates the userscript’s static AdminUserList so liushangchen receives the same administrator privileges as the existing listed accounts. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Deploying xmoj-script-dev-channel with
|
| Latest commit: |
667907e
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://3b3f392a.xmoj-script-dev-channel.pages.dev |
| Branch Preview URL: | https://new-admin.xmoj-script-dev-channel.pages.dev |
There was a problem hiding this comment.
1 issue found across 3 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="Update.json">
<violation number="1" location="Update.json:3957">
P3: This Notes value uses a Markdown link, but the userscript renders Notes through innerHTML (XMOJ.user.js lines 4238 and 5934), so users will see the literal text `[liushangchen](https://www.xmoj.tech/userinfo.php?user=liushangchen)` instead of a clickable link. Every other entry in Update.json uses HTML or plain text; use an HTML anchor here.</violation>
</file>
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
| "Description": "Add 'liushangchen' to AdminUserList" | ||
| } | ||
| ], | ||
| "Notes": "Add [liushangchen](https://www.xmoj.tech/userinfo.php?user=liushangchen) to AdminUserList" |
There was a problem hiding this comment.
P3: This Notes value uses a Markdown link, but the userscript renders Notes through innerHTML (XMOJ.user.js lines 4238 and 5934), so users will see the literal text [liushangchen](https://www.xmoj.tech/userinfo.php?user=liushangchen) instead of a clickable link. Every other entry in Update.json uses HTML or plain text; use an HTML anchor here.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. At Update.json, line 3957:
<comment>This Notes value uses a Markdown link, but the userscript renders Notes through innerHTML (XMOJ.user.js lines 4238 and 5934), so users will see the literal text `[liushangchen](https://www.xmoj.tech/userinfo.php?user=liushangchen)` instead of a clickable link. Every other entry in Update.json uses HTML or plain text; use an HTML anchor here.</comment>
<file context>
@@ -3944,6 +3944,17 @@
+ "Description": "Add 'liushangchen' to AdminUserList"
+ }
+ ],
+ "Notes": "Add [liushangchen](https://www.xmoj.tech/userinfo.php?user=liushangchen) to AdminUserList"
}
}
</file context>
What does this PR aim to accomplish?:
Add liushangchen to AdminUserList.
How does this PR accomplish the above?:
Trivial.
By submitting this pull request, I confirm the following:
git rebase)Summary by Sourcery
Add
liushangchento the administrator list and release the change as version 4.0.1.New Features:
liushangchenas an administrator in the userscript.Build:
Chores:
Summary by cubic
Adds
liushangchento theAdminUserListso the script treats that account as an administrator, and bumps the userscript and package versions to 4.0.1 with matching update metadata.Written for commit 667907e. Summary will update on new commits.