Skip to content

Conversation

@K232
Copy link
Contributor

@K232 K232 commented Sep 23, 2020

As Web API Controller Class does not exist, I took Controller Class - empty. It has no templated class methods so I always received a 404 :-( Therefore I've added a HTTPGet so I get the 200 :-) Additionally it does not work for me with localhost:5001 but with the port specified in ssl-port.

As Web API Controller Class does not exist, I took Controller Class - empty. It has no templated class methods so I always received a 404 :-( Therefore I've added a HTTPGet so I get the 200 :-) Additionally it does not work for me with localhost:5001 but with the port specified in ssl-port.
@ghost
Copy link

ghost commented Sep 23, 2020

CLA assistant check
All CLA requirements met.

@ktoliver ktoliver added the aq-pr-triaged tracking label for the PR review team label Sep 23, 2020
@ktoliver ktoliver added documentation Improvements or additions to documentation mobile-apps labels Sep 23, 2020
@mikeparker104
Copy link
Contributor

mikeparker104 commented Sep 24, 2020

Thank you @K232 and @ktoliver for your thorough input. The steps provided were for VS Mac specifically. The proposed changes include reworking existing steps based on VS 2019. However, I think there is a broader work item to more clearly highlight that prerequisite and to call out key differences making it easier to follow in VS 2019. @ktoliver perhaps this can be raised as a separate issue addressing friction for those following with VS 2019 based on feedback from all related issues such as #300.

@K232
Copy link
Contributor Author

K232 commented Sep 24, 2020

Hi @mikeparker104, no idea how it looks on VS for Mac. Prerequisites say it's for Mac or Windows so I took windows. I don't know whether wording is different on Mac and Windows (which would be surprising?). If so, maybe add information about the naming on windows. I just tried to follow the instructions and wanted to let you know of the differences I had found on Windows.

@mikeparker104
Copy link
Contributor

Hi @K232. Really appreciate you pointing out these challenges. I can see how the following wording might be ambiguous: 'The steps provided are for Visual Studio for Mac but it's possible to follow along using Visual Studio 2019'. Making this clearer and adding further information specific to Windows, where it makes sense, will make it easier to follow. Thank you again for taking the time to share this feedback.

@mikeparker104
Copy link
Contributor

Thanks for the feedback @K232. I've adapted and combined it with other feedback into a single commit.

@ktoliver
Copy link
Contributor

We'll close the PR. Thanks!

@ktoliver ktoliver closed this Oct 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

aq-pr-triaged tracking label for the PR review team documentation Improvements or additions to documentation mobile-apps

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants