Skip to content

feat(Tengoku Fansub): add activity#10657

Open
fkarlsefnii wants to merge 9 commits intoPreMiD:mainfrom
fkarlsefnii:main
Open

feat(Tengoku Fansub): add activity#10657
fkarlsefnii wants to merge 9 commits intoPreMiD:mainfrom
fkarlsefnii:main

Conversation

@fkarlsefnii
Copy link
Copy Markdown

@fkarlsefnii fkarlsefnii commented Apr 5, 2026

Description

Acknowledgements

Screenshots

Proof showing the creation/modification is working as expected

.
Signed-off-by: fkarlsefnii <aqayefmirfariz@icloud.com>
Signed-off-by: fkarlsefnii <aqayefmirfariz@icloud.com>
Copy link
Copy Markdown

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

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.

.
Signed-off-by: fkarlsefnii <aqayefmirfariz@icloud.com>
Signed-off-by: fkarlsefnii <aqayefmirfariz@icloud.com>
Signed-off-by: fkarlsefnii <aqayefmirfariz@icloud.com>
Signed-off-by: fkarlsefnii <aqayefmirfariz@icloud.com>
Signed-off-by: fkarlsefnii <aqayefmirfariz@icloud.com>
Signed-off-by: fkarlsefnii <aqayefmirfariz@icloud.com>
Signed-off-by: fkarlsefnii <aqayefmirfariz@icloud.com>
@@ -0,0 +1,42 @@
{
"$schema": "https://schemas.premid.app/metadata/1.7",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
"$schema": "https://schemas.premid.app/metadata/1.7",
"$schema": "https://schemas.premid.app/metadata/1.16",

"url": "tengoku-fansub.com",
"version": "1.0.0",
"logo": "https://i.imgur.com/TAYvhp6.png",
"thumbnail": "https://i.imgur.com/TAYvhp6.png",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thumbnail should be a landscape promotional image or screenshot, not the logo

Comment on lines +20 to +21
"iframe": false,
"readLogs": false,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
"iframe": false,
"readLogs": false,

Unnecessary

Comment on lines +20 to +22
const showEpisode = await presence.getSetting<boolean>('showEpisode')
const showTimestamp = await presence.getSetting<boolean>('showTimestamp')
const showButtons = await presence.getSetting<boolean>('showButtons')
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Combine using Promise.all

const { pathname, href } = document.location

const presenceData: PresenceData = {
type: 3, // ActivityType.Watching
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Use the ActivityType enum from the premid package instead


presenceData.details = anime || 'Anime'
presenceData.state = 'Anime səhifəsini araşdırır'
presenceData.smallImageKey = 'browse'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Use image URL instead of discord asset key


presenceData.details = query ? `"${query}" axtarır` : 'Arxivi axtarır'
presenceData.state = 'Tengoku Fansub'
presenceData.smallImageKey = 'search'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You can use Assets.Search from the Assets enumeration in the PreMiD package

if (pathname.startsWith('/profile')) {
presenceData.details = 'Profilinə baxır'
presenceData.state = 'Tengoku Fansub'
presenceData.smallImageKey = 'browse'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Use image URL instead of discord asset key

if (pathname.startsWith('/marketplace')) {
presenceData.details = 'Mağazaya baxır'
presenceData.state = 'Tengoku Fansub'
presenceData.smallImageKey = 'browse'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Use image URL instead of discord asset key

// Ana səhifə / digər
presenceData.details = 'Tengoku Fansub'
presenceData.state = pathname === '/' || pathname === '' ? 'Ana səhifədə' : 'Sayta baxır'
presenceData.smallImageKey = 'logo'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Use image URL instead of discord asset key

@fkarlsefnii fkarlsefnii requested a review from skullysmods April 6, 2026 07:27
@theusaf theusaf changed the title feat: add Tengoku Fansub presence feat(Tengoku Fansub): add activity Apr 6, 2026
Copy link
Copy Markdown
Author

@fkarlsefnii fkarlsefnii left a comment

Choose a reason for hiding this comment

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

.

Copy link
Copy Markdown
Contributor

@skullysmods skullysmods left a comment

Choose a reason for hiding this comment

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

You haven't fixed the issues I reported earlier

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.

4 participants