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

PDS: Fix API Request forwarding for BSky APIs #52

Merged
merged 8 commits into from
Feb 10, 2025

Conversation

TheRipperoni
Copy link
Contributor

@TheRipperoni TheRipperoni commented Feb 7, 2025

Implementation resolves issues related to atproto-proxy calls.

For a list of calls tested to be confirmed now correctly working

  • Update Read Notifations
  • All BSky Chat APIs (Both GET and POST requests)

Have removed the chat mod as this is currently not being reached, and leveraging atproto-proxy lets rsky be more agnostic to the app server.

Addresses
#18

Copy link
Member

@rudyfraser rudyfraser left a comment

Choose a reason for hiding this comment

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

Requested a couple changes and open questions.

@TheRipperoni TheRipperoni changed the title Fix API Request forwarding for BSky APIs PDS: Fix API Request forwarding for BSky APIs Feb 9, 2025
Copy link
Contributor Author

@TheRipperoni TheRipperoni left a comment

Choose a reason for hiding this comment

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

Reviewed Request Changes

Copy link
Contributor Author

@TheRipperoni TheRipperoni left a comment

Choose a reason for hiding this comment

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

Updated reviews

@rudyfraser rudyfraser merged commit 341d7da into blacksky-algorithms:main Feb 10, 2025
4 checks passed
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