nodeSolidServer/solid-auth-client

README Logging in snippet should include the key "popupUri"

dprat0821 opened this issue · 3 comments

In the Logging in session of README.md, there is a snippet of popupLogin with the following statement:
session = await solid.auth.popupLogin({ popupUri });
Got blank popup page with it. After checking the source code I tried and found that it should be
session = await solid.auth.popupLogin({ popupUri: popupUri });

My envrionment: MacOS, Angular 7.0.4, Node: 8.12.9. OSX 10.14.2.

Given that

{ popupUri }

is syntactic sugar for

{ popupUri: popupUri }

I strongly suspect the error is somewhere else. Can you verify?

I see. My original code was like this since it's a member variable in my code:
session = await solid.auth.popupLogin({ popupUri: this.popupUri });
Great to learn this syntax sugar. Still suggest maybe we can put a note there in case avoid people who don't know it spend hours troubleshooting :)

Happy to hear it works 🙂