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

Failed to read OpenligaDB Bl1 programmatically #65

Closed
d002902 opened this issue Jul 2, 2023 · 10 comments
Closed

Failed to read OpenligaDB Bl1 programmatically #65

d002902 opened this issue Jul 2, 2023 · 10 comments
Assignees
Labels

Comments

@d002902
Copy link

d002902 commented Jul 2, 2023

The Browser delivers the expected data:
https://api.openligadb.de/getmatchdata/bl1/2023
https://api.openligadb.de/getavailableteams/bl1/2023

but not programmatically.
Are there changes necessary? (Last 2022/2023 worked fine.)
Please advise.

Thanks and regards Michael

@KaterMikesch
Copy link

Same here.

@meyomey
Copy link

meyomey commented Jul 2, 2023

also here for bl1.
bl2 is working.

@msiggi msiggi self-assigned this Jul 3, 2023
@msiggi
Copy link
Contributor

msiggi commented Jul 3, 2023

Please, check the upper and lower case of the objects. Yes, for several reasons we had to change this spelling

For example, the date of the match is now "matchDateTime", former "MatchDateTime".

I´m very sorry, if you have trouble with this...

@d002902
Copy link
Author

d002902 commented Jul 3, 2023 via email

@meyomey
Copy link

meyomey commented Jul 3, 2023 via email

@d002902
Copy link
Author

d002902 commented Jul 3, 2023 via email

@msiggi
Copy link
Contributor

msiggi commented Jul 3, 2023

@d002902, @meyomey : Sorry, but I can not reproduce any of your issues. It seems, the problem is related to your clients, because:

  1. As you mentioned, in browser all values where returned correct.
  2. I added angular-with-Results-complete-league.html, please try it as a minimal api-client. In my environment all matches returned as expected. Please take a look in the code, is my request maybe partly different from yours?

@d002902
Copy link
Author

d002902 commented Jul 4, 2023 via email

@KaterMikesch
Copy link

Works for me with the new Base-URL. Thanks. The breaking change (lower-upper-case) happened a while ago in the midst of the season. I already changed that.

All in all much better now with the Swagger based API. The transition was not really 100% smooth but it is a really, really nice project anyway. Thanks a ton.

@msiggi
Copy link
Contributor

msiggi commented Jul 4, 2023

The transition was not really 100% smooth

I know about, sorry again.

but it is a really, really nice project anyway. Thanks a ton.

Thank you ;-)

@msiggi msiggi closed this as completed Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants