Fix "undefined" message

uetchy / Polyglot

🌏 The missing Safari extension that translates selected text into your native language.

Other

482 stars 44 forks source link

Fix "undefined" message #45

Closed devemio closed 5 years ago

devemio commented 5 years ago

Guys, Happy New Year 🎄

It fixes #41 and #42.

Looks like Goole has a limitation on the number of api calls for ip. Instead of the undefined message, a more understandable message will be displayed.

Now it looks like this:

Maybe the message text need to be corrected, if you have any ideas please let me know.

uetchy commented 5 years ago

Thank you for the PR! I'll merge it if you fix the commit tree issue which includes redundant unrelated commits. Maybe only we need is abc9acee4f1679c953ffbf185fb624720c10f063.

devemio commented 5 years ago

@uetchy, I've pushed new changes. Now there is only one commit.

uetchy commented 5 years ago

Nice! merged🎉

hackerwin7 commented 5 years ago

when dose this feature release?

uetchy commented 5 years ago

released now: https://github.com/uetchy/Polyglot/releases/tag/v2.6.0