Skip to content

feat(NextToppers): add activity - #11047

Merged
theusaf merged 15 commits into
PreMiD:mainfrom
klaze-125:main
Aug 3, 2026
Merged

feat(NextToppers): add activity#11047
theusaf merged 15 commits into
PreMiD:mainfrom
klaze-125:main

Conversation

@klaze-125

Copy link
Copy Markdown
Contributor

Added new presence activity for NextToppers (nexttoppers.com). Compiled cleanly and tested locally.

@dimerp

dimerp Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

🖼️ Activity asset preview

I fetched these assets so you don't have to. My hobbies include embedding images and silently judging diffs.

NextToppers (websites/N/NextToppers) — creation

Preview Source URL
logo https://i.imgur.com/9YKfQ6M.png
thumbnail https://i.imgur.com/NfGW7mM.png

Beep boop — I refresh this comment on every push, so no need to scroll. 🤖

@klaze-125 klaze-125 changed the title Add NextToppers activity feat(NextToppers): add activity Jul 26, 2026

@skullysmods skullysmods left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The activity is of low quality. Add support for other pages. + Fix these errors + upload proof of working it

Comment thread websites/N/NextToppers/metadata.json
Comment thread websites/N/NextToppers/metadata.json Outdated
Comment thread websites/N/NextToppers/metadata.json Outdated
@klaze-125

Copy link
Copy Markdown
Contributor Author
image image

Updated metadata.json with $schema v1.17, corrected regExp, and set a distinct landscape thumbnail URL.
Updated presence.ts with dynamic multi-page routing support.
Attached proof screenshot of the presence working locally on Discord.

@klaze-125
klaze-125 requested a review from skullysmods July 27, 2026 11:02

@github-advanced-security github-advanced-security AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ESLint found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

Comment thread websites/N/NextToppers/metadata.json Fixed
Comment thread websites/N/NextToppers/metadata.json Fixed
Comment thread websites/N/NextToppers/presence.ts Fixed
Comment thread websites/N/NextToppers/metadata.json Outdated
Comment thread websites/N/NextToppers/metadata.json Outdated
Comment thread websites/N/NextToppers/presence.ts Outdated
Comment thread websites/N/NextToppers/presence.ts Outdated
@klaze-125
klaze-125 requested a review from skullysmods July 27, 2026 16:09
Comment thread websites/N/NextToppers/presence.ts Fixed
Comment thread websites/N/NextToppers/presence.ts
@skullysmods
skullysmods requested a review from a team July 28, 2026 13:32
Bas950
Bas950 previously requested changes Jul 28, 2026

@Bas950 Bas950 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

show some more dynamic data, like which course etc.

auto-merge was automatically disabled July 29, 2026 15:04

Head branch was pushed to by a user without write access

klaze-125 and others added 9 commits July 29, 2026 19:04
Signed-off-by: Klaze <kailassajilal125@gmail.com>
Co-authored-by: Skully <117316808+skullysmods@users.noreply.github.com>
Signed-off-by: Klaze <kailassajilal125@gmail.com>
Signed-off-by: Klaze <kailassajilal125@gmail.com>
Signed-off-by: Klaze <kailassajilal125@gmail.com>
Signed-off-by: Klaze <kailassajilal125@gmail.com>
Co-authored-by: Skully <117316808+skullysmods@users.noreply.github.com>
Signed-off-by: Klaze <kailassajilal125@gmail.com>
Signed-off-by: Klaze <kailassajilal125@gmail.com>
Signed-off-by: Klaze <kailassajilal125@gmail.com>
Signed-off-by: Klaze <kailassajilal125@gmail.com>
@klaze-125

Copy link
Copy Markdown
Contributor Author

Updated presence.ts to dynamically extract page headings for course/lecture names and added live video player timestamps.

@klaze-125
klaze-125 requested a review from Bas950 July 29, 2026 16:04
@theusaf
theusaf enabled auto-merge (squash) July 30, 2026 18:22
@klaze-125

Copy link
Copy Markdown
Contributor Author

@Bas950 @theusaf Requesting a review update or dismissal on the stale changes-requested block. Dynamic course titles and video timestamps have been implemented, tested, and approved.

@skullysmods
skullysmods dismissed Bas950’s stale review August 3, 2026 08:40

changes were made

@theusaf
theusaf merged commit ffc6bc4 into PreMiD:main Aug 3, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

5 participants