-
Notifications
You must be signed in to change notification settings - Fork 14
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
Feature/friends subsystem #32
Open
MikhailSorokin
wants to merge
37
commits into
Ruhrpottpatriot:master
Choose a base branch
from
MikhailSorokin:Feature/FriendsSubsystem
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 19 commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
549dbd7
Compiled code with fixed identity and user interface calls. Will need…
fc20ef4
Found a bug where the query was never being triggered, so one could n…
abrarsaleh bfcd213
Queries now complete properly - first problem fix. Am having trouble …
abrarsaleh 339661b
Fixed the final error in user info queries - which was getting the na…
abrarsaleh f636990
Commented out presence as it throws a compile error.
abrarsaleh 4b116f0
Did cleanup of code and removed friends interface from here for the s…
abrarsaleh 536ff10
Some refactoring changes with friend invite stuff.
MikhailSorokin d07e70a
Addressed all commments for PR besides merge conflict.
MikhailSorokin ea3b31d
Addressed merge conflict and changed back to 'ToProdcutUserId' lol. S…
MikhailSorokin 3e28ba8
Friend queries now work and refactored. NEXT STEP: Get subscription e…
MikhailSorokin 1655913
Done how I think presence should work and should fix a subscribe quer…
MikhailSorokin 38cdd77
Fix file spelling error
Ruhrpottpatriot 1f4f76a
Updated with latest master. Added in several fixes to the FUniqueNetI…
MikhailSorokin 5879bb8
Merge branch 'Bug/UserInfoQuery' into Feature/FriendsSubsystem
MikhailSorokin b9aa1b0
FIx minor style things
Ruhrpottpatriot 2bc8ee7
Issue fixed with null-termination.
MikhailSorokin 0dc6dee
QueryPresence is now returning a success call but I cannot execute th…
MikhailSorokin 9a51150
Quering presence now works - tested with friends. However, I want to …
MikhailSorokin b2c4954
Presence + friend for subscription events no longer crashes but way t…
MikhailSorokin 2371fa7
Complete friends and presence implementation. For some reason, subscr…
abrarsaleh fe85150
Merged with updated master. This now incorporates all changes of what…
abrarsaleh ce56b12
Compile error fix.
abrarsaleh 131bca2
User query info complete fix.
MikhailSorokin d88b381
Small formatting fixes.
MikhailSorokin 3f96cd6
Merging with latest master.
MikhailSorokin ab2705c
Removal of finished query and check if we have information already ca…
MikhailSorokin a3e5a11
Merging with query user info bug I fixed.
MikhailSorokin 052a6fd
Updating some stuff to do if we are already cached + some refactooring.
MikhailSorokin a8b3e76
Cleaned up - ready for review.
bc20e5a
Cleanup of friend and presence classes.
4a28947
Added check upon testing - account portal seems to query the local us…
13a0060
Removal of lobby and party classes - they weren't meant to be put in …
0a2bc1b
Addressed both requested changes from Ruhrpottpatriot 8/3/2020.
428096c
Addressing style and some format fixes 8/3/20 from Ruhrpottpatriot.
ac92ec0
Updating with Bug/UserInfo query branch.
098c410
Final changes after testing.
cb2c92a
Final tests were conducted on a friends list of 6 people and now work…
MikhailSorokin File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check if you and your team have configured the build process correctly. You shouldn't need the Private part of the include path.