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

Request Season parameter be added to FleaFlicker endpoints #405

Closed
RandalMorris opened this issue Aug 10, 2023 · 6 comments
Closed

Request Season parameter be added to FleaFlicker endpoints #405

RandalMorris opened this issue Aug 10, 2023 · 6 comments

Comments

@RandalMorris
Copy link
Contributor

When using ff_franchises for example, the FetchLeagueStandings endpoint accepts season as a parameter, but the package isn't passing it from the connection dataset. It defaults to current season so even if the connection data has 2022, it is pulling 2023

@tanho63
Copy link
Member

tanho63 commented Aug 10, 2023

Hmm. This looks like a pretty straightforward addition- do you want to give it a go in a PR?

@RandalMorris
Copy link
Contributor Author

I changed the R/flea_franchises.R to fleaflicker_getendpoint("FetchLeagueStandings", season = conn$season, league_id = conn$league_id, sport = "NFL"). Still pulls 2023, I even manually set it to 2022.

When I create the function locally, it works so i am missing something.

@tanho63
Copy link
Member

tanho63 commented Aug 10, 2023

hmm. may need to restart R to make sure it’s not being cached

@RandalMorris
Copy link
Contributor Author

That did it, I thought i had tried that lol.

I will submit a PR soon. I am going through them to add that where it can be used.

@tanho63 tanho63 reopened this Aug 11, 2023
@tanho63
Copy link
Member

tanho63 commented Aug 11, 2023

Gonna keep this open and close it when the appropriate PR is merged into main, see feedback in PR comments

@tanho63
Copy link
Member

tanho63 commented Aug 13, 2023

Resolved in #408

@tanho63 tanho63 closed this as completed Aug 13, 2023
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

No branches or pull requests

2 participants