search

PlayFab / PlayFab-Samples

This repository contains Sample code, Recipes and Video Tutorials demonstrating how to integrate various PlayFab features
https://www.playfab.com
MIT License
208 stars 241 forks source link

Documentation out of date for PlayFab and SignalR setup #47

Open ivandamyanov opened 4 years ago

ivandamyanov commented 4 years ago

Instructions not clear... I created a remote controlled missile execution platform.

Jokes aside, first issue is with the connection string for this sample: https://github.com/PlayFab/PlayFab-Samples/tree/master/Samples/CSharp/MessagingClient

"4. Open the local.settings.json file in the PlayFab.SignalRSample project in this repo" That is not enough. After publishing the functions, they throw errors that there's no SignalR connection string. I manually added it from the Azure Portal in the "Configure" section of the function. The error went away. I guess that's related to other documentation being out of date for more or less a year: https://github.com/MicrosoftDocs/azure-docs/issues/28140

The second issue: "Repeat steps 3-10 using Name=Matchmaking Notifications, Event type=playfab.matchmaking.match_found, and Function=MatchFound"

This event is no longer found as a trigger: "playfab.matchmaking.match_found". I found this comment "https://community.playfab.com/questions/35040/cannot-add-rule-for-event-matchmaking-match-found.html"

amccalib commented 4 years ago

I'll take a look at the first issue. As to the second, it looks like the match_found event isn't available to trigger cloud script for all titles. It's actually part of a limited preview. If you want to be added while it's still in preview, let me know.

amccalib commented 4 years ago

I fixed the documentation to account for the first issue. Still waiting on the Azure Function trigger to be made fully public to address the second.