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

feat: friend profiles, protocol update & events improvement #3197

Closed
wants to merge 6 commits into from

Conversation

lorux0
Copy link
Collaborator

@lorux0 lorux0 commented Jan 23, 2025

What does this PR change?

Integrates the latest changes of the friendship protocol: decentraland/protocol#242
Introduces the FriendProfile class which contains the essential information of a friend so it is represented in the UI.
Improves the event bus by renaming the events to a more descriptive manner and adds new events whenever you make any friendship operation.

How to test the changes?

  1. Launch the explorer
  2. Check that the startup flow works as expected

Our Code Review Standards

https://github.com/decentraland/unity-renderer/blob/master/docs/code-review-standards.md

# Conflicts:
#	Explorer/Assets/DCL/PluginSystem/Global/FriendsPlugin.cs
#	Explorer/Assets/Scripts/Global/Dynamic/DynamicWorldContainer.cs
Copy link
Collaborator

@AnsisMalins AnsisMalins left a comment

Choose a reason for hiding this comment

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

The code is fine, but there is, in my opinion, a design flaw. People who are not your friends should not receive events about you. If you reject someone's friend request, that action should be private to you.

@m3taphysics m3taphysics added the clean-build Used to trigger clean build on PR label Jan 27, 2025
@lorux0
Copy link
Collaborator Author

lorux0 commented Jan 28, 2025

Since we should not merge changes until the protocol pr is merged this pr becomes deprecated as we will be including everything in a bigger pr which contains all the friends integration.

@lorux0 lorux0 closed this Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clean-build Used to trigger clean build on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants