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

added league_type = contest to mfl_league #439

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jak3sch
Copy link
Contributor

@jak3sch jak3sch commented Sep 26, 2024

I'm playing in a mfl contest league and thought, it would be usefull to get this type via mfl_league().

For that, I use the API endpoint loadRosters if no keeperType is found. I don't know, how robust this is and if someone is playing a keeper contest league or if this is even possible. But maybe this is better than nothing?

@tanho63
Copy link
Member

tanho63 commented Sep 26, 2024

Hmm. It could be worth reaching out to MFL support to ask if that’s the right way to identify a contest league via API?

@jak3sch
Copy link
Contributor Author

jak3sch commented Sep 27, 2024

It seems like this is the way to go. From the mfl support:

"You can just use the "league" API call, and then check the "loadRosters" variable to check if it is a contest style league. It wouldn't make sense for a contest style league to have the dynasty setting, so that shouldn't be an option that you need to check."

R/mfl_league.R Outdated Show resolved Hide resolved
@jak3sch
Copy link
Contributor Author

jak3sch commented Oct 14, 2024

I added your suggestion

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

Successfully merging this pull request may close these issues.

2 participants