search

monicahq / chandler

The upcoming brand new version of Monica. Not suitable for production use at the moment.
GNU Affero General Public License v3.0
270 stars 12 forks source link

Cannot save address if Location IQ is not set up #340

Open Laptop765 opened 1 year ago

Laptop765 commented 1 year ago

I tried adding an address without having Location IQ set up and received an error:

172.19.0.12 - - [11/Dec/2022:14:03:34 -0800] "POST /vaults/1/contacts/1/addresses HTTP/1.1" 500 1205 "https://paul.contacts.berruti.family/vaults/1/contacts/1/tabs/ways-to-connect" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/16.1 Safari/605.1.15" [2022-12-11 22:04:19] production.ERROR: Env variables are not set for Location IQ {"userId":1,"exception":"[object] (App\\Exceptions\\EnvVariablesNotSetException(code: 0): Env variables are not set for Location IQ at /var/www/html/app/Domains/Contact/ManageContactAddresses/Services/GetGPSCoordinate.php:89)

I can provide a whole call stack but I imagine this is by design during development. If you don't want these kinds of bug reports just yet I'm happy to hold off but wanted to help if it's useful!

Laptop765 commented 1 year ago

Oh it looks like it actually did save and when I refreshed I have 10 entries for all the times I tried clicking.

djaiss commented 1 year ago

Thanks for the bug report! I'll fix that 😄

Laptop765 commented 1 year ago

Are you open to bug reports like this yet? I don't want to add noise for things you already know about and/or are planned for later.

djaiss commented 1 year ago

Are you open to bug reports like this yet? I don't want to add noise for things you already know about and/or are planned for later.

We definitely are. We are soon to release a beta version so we want to know what's working and what's not.

Laptop765 commented 1 year ago

Awesome, thanks for being so responsive!

Laptop765 commented 1 year ago

Also probably related, the page doesn't refresh to show the new address.