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

Retrieve avatar via ActivityPub #142

Merged
merged 3 commits into from
Dec 30, 2022
Merged

Retrieve avatar via ActivityPub #142

merged 3 commits into from
Dec 30, 2022

Conversation

akirk
Copy link
Owner

@akirk akirk commented Dec 30, 2022

Fixes #139.

This will use an avatar from ActivityPub when available:
avatar-new-user

You can also update an avatar from ActivityPub (as it will not (yet?) update the avatar automatically when a user changes it):
update-avatar

@akirk akirk merged commit c9d946f into main Dec 30, 2022
@akirk akirk deleted the activitypub-avatar branch December 30, 2022 03:38
@gbraad
Copy link

gbraad commented Dec 30, 2022

Wanted to try this out, but got an error:

[Thu Dec 29 20:37:59.123047 2022] [fcgid:warn] [pid 340940:tid 3637677836032] [client 45.158.40.164:50940] mod_fcgid: stderr: PHP Fatal error:  Declaration of Activitypub\\Friends_Feed_Parser_ActivityPub::fetch_feed($url) must be compatible with Friends\\Feed_Parser::fetch_feed($url, ?Friends\\User_Feed $user_feed = null) in /home/spotsnel_gbraad/sites/gbraad.nl/blog/wp-content/plugins/activitypub/integration/class-friends-feed-parser-activitypub.php on line 137

@akirk
Copy link
Owner Author

akirk commented Dec 30, 2022

@gbraad thanks for flagging, I tried to fix it in #143.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

User avatars are refering to favicon.ico
2 participants