Skip to content

Add esports-pass extension#17269

Merged
raycastbot merged 23 commits intoraycast:mainfrom
rafaelvicio:ext/esports-pass
Mar 3, 2025
Merged

Add esports-pass extension#17269
raycastbot merged 23 commits intoraycast:mainfrom
rafaelvicio:ext/esports-pass

Conversation

@rafaelvicio
Copy link
Contributor

@rafaelvicio rafaelvicio commented Feb 23, 2025

Description

Esports Pass
A extesnons for esports. League of Legends, VALORANT, CS and more.

  • Show Matches - Show esports matches
  • Show Results - Show esports results
  • Show Tournaments - Show esports tournaments

Screencast

esports-pass-1 esports-pass-2 esports-pass-3

Checklist

- feat: public extension
- feat: change screnshots
- feat: add metadata
- feat: add changelog
- feat: use npm
- feat: fix tournament name
- feat: add tournaments command
- feat: refact commands
- init
@raycastbot raycastbot added the new extension Label for PRs with new extensions label Feb 23, 2025
@raycastbot
Copy link
Collaborator

Congratulations on your new Raycast extension! 🚀

Due to our current reduced availability, the initial review may take up to 10-15 business days

Once the PR is approved and merged, the extension will be available on our Store.

@pernielsentikaer
Copy link
Collaborator

Thanks for your contribution, it could maybe make sense to unify unto one extension, I can see that we have 2/3 games in the store already made by @vimtor

What do you think?

@pernielsentikaer pernielsentikaer self-assigned this Mar 1, 2025
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Added a new Esports Pass extension that allows users to view esports matches, results, and tournaments for games like League of Legends, VALORANT, and Counter-Strike.

  • The CHANGELOG.md entry should use {PR_MERGE_DATE} instead of an actual date (2025-02-23)
  • README.md contains typos: "A extesnons for esports?" should match the title in package.json
  • Missing metadata folder with screenshots for view commands as required for new extensions
  • Consider adding subtitles to commands in package.json to indicate the service
  • Lists in all three commands should use the isLoading prop to avoid empty state flicker
  • Comparison operators should use === instead of == when checking for empty strings

💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!

10 file(s) reviewed, 10 comment(s)
Edit PR Review Bot Settings | Greptile

rafaelvicio and others added 11 commits March 1, 2025 12:44
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
@rafaelvicio
Copy link
Contributor Author

Ready for a new review. @greptileai @greptile-apps @raycastbot

@rafaelvicio
Copy link
Contributor Author

Thanks for your contribution, it could maybe make sense to unify unto one extension, I can see that we have 2/3 games in the store already made by @vimtor

What do you think?

Hi, yes I know about extensions.

But the Esports Pass extension is more connected to our own API. And we have a good roadmap planned for the inclusion of new features and new games.

The next games are: Call of Duty, Dota2, EA Sports FC, LOL Wild Rift, Mobile Legends: Bang Bang, Overwatch, PUBG, Rocket League, King of Glory, Starcraft 2 and Starcraft Brood War

There will be features that are specific to connections with the user's Esports Pass account.

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

(updates since last review)

This PR adds a new Esports Pass extension for viewing esports matches, results, and tournaments for popular games like League of Legends, VALORANT, and Counter-Strike.

  • Type error in /extensions/esports-pass/src/results.tsx where Matche[] is used instead of Match[] in the Schedule interface
  • Missing metadata folder with screenshots for view commands as required for new extensions
  • In /extensions/esports-pass/src/matches.tsx and other files, component naming is inconsistent (e.g., MatcheItem vs expected MatchItem)
  • Consider adding additional games from assets folder (rainbow-six.png and dota.png) to the videogames array
  • API calls should be wrapped in try/catch blocks for better error handling

11 file(s) reviewed, 5 comment(s)
Edit PR Review Bot Settings | Greptile

@pernielsentikaer
Copy link
Collaborator

@rafaelvicio could you have a look?

@rafaelvicio
Copy link
Contributor Author

@pernielsentikaer I think this is super cool extension and we should keep it as a it is. One notable difference is that my extensions uses Riot's API so if something happened to Esports Pass the extension would become obsolete.

Probably makes sense to have both and, at some point, remove my extensions.

I agree with @vimtor , the lolesports API can also count on other championships that are not covered by our data providers. It can also be discontinued at any time, and is a valid risk on both sides.

I believe that these two extensions can coexist. @pernielsentikaer

Copy link
Collaborator

@pernielsentikaer pernielsentikaer left a comment

Choose a reason for hiding this comment

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

Hi 👋

Looks good to me, approved 🔥

@raycastbot raycastbot merged commit c288820 into raycast:main Mar 3, 2025
2 checks passed
@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2025

Published to the Raycast Store:
https://raycast.com/rafaelvicio/esports-pass

@raycastbot
Copy link
Collaborator

🎉 🎉 🎉

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

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

Labels

new extension Label for PRs with new extensions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants