fal-ai / fal

⚡ Fastest way to serve open source ML models to millions
https://fal.ai/docs
Apache License 2.0
549 stars 47 forks source link

fix(auth): always try to open browser but also show url and code #132

Closed efiop closed 7 months ago

efiop commented 7 months ago

The prompt is unnecessary, we should just try opening a browser automatically.

Currently the URL tells me to check the code in CLI, which is not shown to me if webbrowser.get() was able to find a browser. Also then open_new_tab is not really guaranteed to work, so we are better off just always showing the URL as well.

Screenshot 2024-03-21 at 01 25 41

Also it is obvious that the url is a bit too long to comfortably transfer to another device by hand, but I do understand that it is a temporary technical limitation. Just trying to make things clearer from cli side as much as possible.

linear[bot] commented 7 months ago

FEA-2132 auth: try to open browser by default