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

Add RetroAchievements plugin #1620

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

LuanRoger
Copy link

@LuanRoger LuanRoger commented Jun 11, 2024

I've created the RetroAchievements plugin for metrics, using the RetroAchievements's official API.
It follows the style and layout of the Steam plugin, and here it's what I have done so far. I'm opening the PR just to track the checks or feedback.

I've seen that metrics will launch the 4.0 soon, but since there's no release date to it, the changes could be merged and I will do the migrations by my self when the new version's out, fixing any breaking changes and migrating other aspects to fit the new specs.

Render output

RetroAchievement output

This output was generated by my metrics action, using this brench as reference.

Ready to merge?

  • I have correctly filled metadata.yml
    • name is set with an unused emoji and plugin name
    • category is set to community
    • examples contains links towards a rendered output hosted by you
    • authors contains my GitHub username
    • supports list which account type are supported
    • scopes are correctly listed with their associated names on GitHub (leave an empty array if not applicable)
    • inputs are correctly filled
  • I have implemented my plugin
    • index.mjs respects the plugins guidelines
    • Hide achievements when showachievements is no
    • Implement the template (only classic is available)
  • I have tested my plugin locally
    • tests/mocks ... have been created
    • app.placeholder.js has been updated for preview from web instances
    • examples.yml contains workflows examples (at least one is required)
      • skip: true has been set for prod attribute in each test
    • npm run linter yields no errors
  • I have documented my plugin
    • README.md eventually describes complex setup or options (if applicable)
    • Show how to get the Web API Key

@LuanRoger LuanRoger marked this pull request as ready for review June 14, 2024 12:03
Copy link
Contributor

This pull request seems to be inactive and will be automatically closed in a few days if it remains without any activity.

@jayvdb
Copy link
Contributor

jayvdb commented Sep 13, 2024

Not reviewed yet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants