search

sebastienblanc / geolocation

Geolocation grails plugin
http://www.grails.org/plugin/geolocation
Other
13 stars 8 forks source link

404 error returned by ajax call which sets the geoposition #5

Closed gauravchauhan closed 12 years ago

gauravchauhan commented 12 years ago

Looking at console on Chrome. I got this : "Failed to load resource: the server responded with a status of 404 (Not Found)"

On Firefox with firebug add on installed : "NetworkError: 404 Not Found - http://localhost:8080/testGeoLocOnGrails2/geolocation/setGeoPosition"

Even though this is harmless but it doesn't gives a good impression to end user. We can render success from the action to stop getting these errors.

I can submit a pull request in my next commit or you can check-it in yourself.

sebastienblanc commented 12 years ago

Yes please make a pull request ! Thanks again

gauravchauhan commented 12 years ago

Committed the fix.

gauravchauhan commented 12 years ago

Issue must have been fixed by following commit :

https://github.com/sebastienblanc/geolocation/commit/9bf695e348b9ccc5ff2ed3ef58cd08a0acea5dd8

gauravchauhan commented 12 years ago

I will verify it a new machine tomorrow and update here.

gauravchauhan commented 12 years ago

Fixed