PlayFab / PlayFabMultiplayerUnreal

PlayFab Online Subsystem Plugin for Unreal Engine
https://learn.microsoft.com/en-us/gaming/playfab/features/multiplayer/networking/party-unreal-engine-oss-overview
39 stars 35 forks source link

Breaking repo naming consistency #28

Closed dylanh724 closed 1 year ago

dylanh724 commented 1 year ago

https://learn.microsoft.com/en-us/gaming/playfab/features/multiplayer/networking/party-unreal-engine-oss-quickstart#initial-setup

Initial setup

Copy the OnlineSubsystemPlayFab (!) folder ...

Expected: PlayFabMultiplayerUnreal

Must've been renamed. Not a huge deal, but a small issue in a big chain of relating bigger issues that growingly get more frustrating.

There are already so many inconsistencies between the SDK names, API names, repo names (the Unreal MP SDK has about 5 names in the docs, causing your users to endlessly look through your doc link rabbit holes for SDKs where only 1 of 5 actually exists), [...].

That said, it's probably about time to start cracking down on those -- it only confuses your users. You folks really need a professional UX tech doc writer familiar with PF to go in and just fix these things en masse. That will only bandaid the issues, however, if doc-updating practices remain as sterile as it has been.

nassosterz-ms commented 1 year ago

Hello,

Thank you for opening this issue. Could you clarify further what do you mean by:

Expected: PlayFabMultiplayerUnreal

Are you referring to our UE plugin being named PlayFabMultiplayerUnreal to align with the name of this repo?

Every issue brings valuable feedback to us, so appreciate you taking the time to open this.

dylanh724 commented 1 year ago

Hello,

Thank you for opening this issue. Could you clarify further what do you mean by:

Expected: PlayFabMultiplayerUnreal

Are you referring to our UE plugin being named PlayFabMultiplayerUnreal to align with the name of this repo?

Every issue brings valuable feedback to us, so appreciate you taking the time to open this.

Correct - the instructions say to move the dir named OnlineSubsystemPlayFab that does not exist.

nassosterz-ms commented 1 year ago

I see.

The instructions indeed point to a stale name, we will update this accordingly!

dylanh724 commented 1 year ago

I see.

The instructions indeed point to a stale name, we will update this accordingly!

Cheers, and please do consider more doc updates! For example, the entire Client/ and Admin/ Apis were deprecated two years ago without even saying anything. Please read the following with a factual tone, as it may seem aggressive but is more so simply saddening due to the experience I've had:

What gets even worse is your Unreal example (the ONLY example, may I add) starts you off wrong immediately down the deprecated Client/ path, forcing me to implement deprecated and even breaking (below) obsolete apis that I now need to recode due to doc negligence.

Client/Matchmaking is even completely obsolete since GameMode cannot be configured in newer accounts without any notice. To make things more confusing, Matchmaker is ALSO obsolete. Why would there even be 2 matchmaking modules and both obsolete?

But wait, there's a third Multiplayer/Matchmaking module that is the correct one... But is completely missing event callbacks that, for some reason, is now in a completey DIFFERENT sdk (unlike the other 2 apis), yet the docs are so scattered it's impossible to know this. That means you have FOUR matchmaking apis, of which 2 are obsolete (broken due to GameMode) and 1 completely hidden.

Then that different SDK for matchmaking I mentioned? The docs call it 5 different names, friend. Multiplayer SDK. Game SDK. Client SDK. Matchmaking SDK. OSS SDK. There's already doubt since there are 4 matchmaking apis and 2 are broken - then had even more doubt if I even had the right one. The OSS SDK or whatever it's called isn't in the API docs, either, so it's just a guessing game from the limited and conflicted info I have.

This experience has been rough. Very rough. Only because documentation and guides are 2 years stale despite the numerous changes within this time.

Please. Add not only tech doc writers, but UX writers. Your users deserve this.

dylanh724 commented 1 year ago

I see.

The instructions indeed point to a stale name, we will update this accordingly!

Hey, actually... this isn't minor -- it's breaking. I spent a great deal of time researching why my plugin isn't being recognized. Turns out! The .uplugin and the dir it's wrapped in MUST be the same name. Since it's not the same name, it didn't work.

dylanh724 commented 1 year ago

image

On top of that, the old name seems to actually be more-proper to match the other OSS'.

nassosterz-ms commented 1 year ago

We will add documentation to inform that the directory has to be renamed from PlayFabMultiplayerUnreal to OnlineSubsystemPlayFab, I do not think we will change the name of the repo.

nassosterz-ms commented 1 year ago

Documentation fix should be live soon.