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

Fix: (2024.02.02) Update API Endpoint Paths, Domain, and Datetime Handling #198

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

Conversation

EcchiClone
Copy link

2024.02.02

  • Updated the domain string from ".party" to ".su".
  • Revised API endpoint paths in the source code from .../api/... to .../api/v1/... to align with changes in the API server's address.
  • Modified the format_time_by_type function for proper handling of ISO 8601 datetime strings, following adjustments to the API server.
  • isWorking on Python 3.9.0

about tmi

image
It's working perfectly now! (but coomer is so slow to download '-`)

Thank you for your project 🥰

- Updated the domain string from ".party" to ".su".
- Revised API endpoint paths in the source code from .../api/... to .../api/v1/... to align with changes in the API server's address.
- Modified the `format_time_by_type` function for proper handling of ISO 8601 datetime strings, following adjustments to the API server.
@Tosh0kan
Copy link

Tosh0kan commented Feb 2, 2024

Yo man. Thanks a bunch for the fix! I think there still some problems though, because when I try to use it with this pull request, it tries to increment the posts by 25, instead of 50, which ends up causing problems with the API

EDIT

Fixed by going to lines 179 and 181 of main.py and changing the value of 25 to 50

(Commit message and Description translated by GPT-4)

This commit addresses feedback received from Tosh0kan regarding an issue encountered when integrating with a specific pull request. The problem was due to the post increment value being set to 25, which conflicted with the expected value of 50 for proper API functionality. By updating the values on lines 179 and 181 of `main.py` from 25 to 50, this commit resolves the issue and ensures smoother API interactions.

**Changes made:**
- Updated post increment value from 25 to 50 on lines 179 and 181 of `main.py`.

**Acknowledgements:**
- Special thanks to Tosh0kan for identifying the issue and suggesting the precise fix needed to enhance our project's compatibility and performance.
@EcchiClone
Copy link
Author

EcchiClone commented Feb 4, 2024

Yo man. Thanks a bunch for the fix! I think there still some problems though, because when I try to use it with this pull request, it tries to increment the posts by 25, instead of 50, which ends up causing problems with the API

EDIT

Fixed by going to lines 179 and 181 of main.py and changing the value of 25 to 50

Thank you for repair code!

@Tosh0kan
Copy link

Tosh0kan commented Feb 4, 2024

No problem, and sorry I didn't send a proper file

gy6221 and others added 2 commits February 17, 2024 00:06
- Update favourite API url
- Fix datetime parsing to fit the timestamp format in creators.json
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.

3 participants