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.

Sign up for GitHub

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

Jump to bottom

posting to blue sky working #3004

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

posting to blue sky working #3004

wants to merge 27 commits into from

Conversation

krrish-sehgal
Copy link
Contributor

@krrish-sehgal krrish-sehgal commented Nov 28, 2024 •
edited
Loading

Fixes #2955

Solution Description:

  1. An option for the user with mentor tag to approve an Activity to get posted on bluesky and it gets immediately posted.
  2. Also if the post is matching the like thresholds : like>=3 and dislikes < 3 , then it gets automatically posted
  3. New dependency : atproto = "^0.0.55" as mentioned here (https://docs.bsky.app/docs/get-started)

Implementation details :

  1. Implemented the complete like and dislike functionality.
  2. Implemented the automatic posting on blue sky flow.

Bugs:
1.Currently for photos post its showing the alt text instead of the image in blue sky even though photo is going through the endpoint.

Requirements:
In enviroment variables make sure to add:

BLUESKY_USERNAME=example.bsky.social
BLUESKY_PASSWORD='example#123'

Here's a demo of the working:

Screen.Recording.2024-11-29.at.12.43.13.AM.mov

github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 28, 2024
View reviewed changes
website/views/organization.py Fixed Show fixed Hide fixed
website/views/organization.py Fixed Show fixed Hide fixed
website/views/organization.py Fixed Show fixed Hide fixed
@krrish-sehgal
Copy link
Contributor Author

@DonnieBLT sir , Please feel free to review and suggest any changes.
Thanks.

@krrish-sehgal
Copy link
Contributor Author

krrish-sehgal commented Nov 28, 2024 •
edited
Loading

For the failing test case ...
Since it has a new dependency atproto = "^0.0.55", its generating a new poetry.lock.
So is it necessary for me to commit my own poetry.lock file as well or should i make changes in actions workflow and add this line "poetry lock --no-update".

@krrish-sehgal
Copy link
Contributor Author

@DonnieBLT sir , had one pending doubt here, regarding the regenertation of the lock file.
Thanks.

For the failing test case ... Since it has a new dependency atproto = "^0.0.55", its generating a new poetry.lock. So is it necessary for me to commit my own poetry.lock file as well or should i make changes in actions workflow and add this line "poetry lock --no-update".

@krrish-sehgal
Copy link
Contributor Author

@DonnieBLT sir, is this good for merge?

DonnieBLT
DonnieBLT reviewed Dec 1, 2024
View reviewed changes
website/models.py Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DonnieBLT DonnieBLT Awaiting requested review from DonnieBLT

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Post new content to blue sky
2 participants
@krrish-sehgal @DonnieBLT