Next

Closes #2197 (closed) #2087 (closed) #1835 (closed) and half of #2082

Testing

I think it'd be best to spin this up on a subdomain connected to live data, like revolver; so that it can be tested on the live site.

This makes changes to avatars site-wide, user avatars, and group avatars.

To test:

1. Create a new account.
2. Upload an avatar - check it updates everywhere.
3. Create a comment on a post with current avatar.
4. Change avatar.
5. Create a second comment on a post, avatar on all comments should match current avatar of channel.

added scoped labels

added 1 commit

• 631d2347 - Added docs and removed an unused import.

Compare with previous version

changed the description

added 1 commit

• 908ebf3f - Changed service encapsulation for use in template

Compare with previous version

added scoped label and automatically removed label

added 1 commit

• f5118c96 - Refactored and got rid of account switching bug

Compare with previous version

added scoped label and automatically removed label

resolved all threads

resolved all threads

After login, I no longer have an avatar on the topbar .

added scoped label and automatically removed label

Appears to be a change detection issue.

mentioned in merge request engine!386 (merged)

Comment avatars should match current avatar not avatar when comment was made.

changed the description

added 1 commit

• 8871491e - Updated to account for session switching

Compare with previous version

added 1 commit

• e169b919 - Noticed error

Compare with previous version

added 64 commits

• e169b919...9cb3a5b1 - 62 commits from branch master
• 8565bdfa - Merge branch 'master' of gitlab.com:minds/front into fix/avatar-change-detection-2197
• 7d74b971 - Initialize observable at creation

Compare with previous version

approved this merge request

merged

mentioned in commit 9e90087b