Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(OG-Network): add presence #8366

Closed
wants to merge 6 commits into from
Closed

Conversation

John-Fries-J
Copy link

Description

Acknowledgements

Screenshots

Proof showing the creation/modification is working as expected

websites/O/OG Network/presence.ts Outdated Show resolved Hide resolved
websites/O/OG Network/presence.ts Outdated Show resolved Hide resolved
websites/O/OG Network/metadata.json Outdated Show resolved Hide resolved
Comment on lines +67 to +72
presenceData.buttons = [
{
label: "View",
url,
},
];
Copy link
Contributor

Choose a reason for hiding this comment

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

Please read our guideliness

Presences that use buttons should follow extra requirements:
Redirects to main page are prohibited.
Promoting websites by them is prohibited.
They can't display information you couldn't fit in other fields.
Redirecting directly to audio/video stream is prohibited.

Copy link
Member

Choose a reason for hiding this comment

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

If using buttons, it should ideally be used on specific interesting routes rather than a catch-all button


presence.setActivity(presenceData);

if (!startTime) startTime = Date.now();
Copy link
Contributor

Choose a reason for hiding this comment

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

What

websites/O/OG Network/presence.ts Outdated Show resolved Hide resolved
logo: "https://www.og-network.net/assets/logo.png",
};

let startTime: number;
Copy link
Contributor

Choose a reason for hiding this comment

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

no reason to use this after my changes.

Copy link
Member

Choose a reason for hiding this comment

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

startTime is only assigned, not really used. It can be removed.

John-Fries-J and others added 3 commits May 9, 2024 18:51
Co-authored-by: Dark_Ville <42322979+DarkVillager@users.noreply.github.com>
Signed-off-by: John Fries <w.bababooie@gmail.com>
Co-authored-by: Dark_Ville <42322979+DarkVillager@users.noreply.github.com>
Signed-off-by: John Fries <w.bababooie@gmail.com>
Co-authored-by: Dark_Ville <42322979+DarkVillager@users.noreply.github.com>
Signed-off-by: John Fries <w.bababooie@gmail.com>
websites/O/OG Network/presence.ts Outdated Show resolved Hide resolved
Co-authored-by: Dark_Ville <42322979+DarkVillager@users.noreply.github.com>
Signed-off-by: John Fries <w.bababooie@gmail.com>
Copy link
Member

@theusaf theusaf left a comment

Choose a reason for hiding this comment

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

Also, resolve the automated checks

],
"version": "1.0.0",
"logo": "https://i.imgur.com/6jC2EYO.png",
"thumbnail": "https://www.og-network.net/assets/logo.png",
Copy link
Member

Choose a reason for hiding this comment

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

The thumbnail should preferably be a wide promotional card or a screenshot if the first is not available.

websites/O/OG Network/presence.ts Show resolved Hide resolved
logo: "https://www.og-network.net/assets/logo.png",
};

let startTime: number;
Copy link
Member

Choose a reason for hiding this comment

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

startTime is only assigned, not really used. It can be removed.

Comment on lines +67 to +72
presenceData.buttons = [
{
label: "View",
url,
},
];
Copy link
Member

Choose a reason for hiding this comment

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

If using buttons, it should ideally be used on specific interesting routes rather than a catch-all button

},
"service": "OG Network",
"description": {
"en": "OG Network website presence"
Copy link
Member

Choose a reason for hiding this comment

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

If possible, use an official description

@theusaf theusaf added the inactive PR has been inactive for a while and will be closed soon label May 21, 2024
@theusaf
Copy link
Member

theusaf commented May 23, 2024

Closing for inactivity. You may open a new PR after resolving the issues mentioned in this one.

@theusaf theusaf closed this May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inactive PR has been inactive for a while and will be closed soon new presence
Development

Successfully merging this pull request may close these issues.

None yet

3 participants