Skip to content
This repository has been archived by the owner on Oct 17, 2022. It is now read-only.

Commit

Permalink
Fixed issue with non-default lat/long fields, and not providing defau…
Browse files Browse the repository at this point in the history
…lts when field names were not specified
  • Loading branch information
mikebronner committed Jun 26, 2019
1 parent 0d711fb commit d4d5eb6
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 3 deletions.
8 changes: 8 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,16 @@ All notable changes to this project will be documented in this file.

The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).

## [Unreleased]

## [0.1.3] - 2019-06-26
### Fixed
- an issue where the detail screen didn't read non-default `latitude` and
`longitude` fields.
- an issue on the detail screen where it didn't apply the default field names
when not provided.

## [0.1.2] - 2019-06-24
### Fixed
- scroll and touch zoom behavior, where the marker would move off center when
Expand Down
2 changes: 1 addition & 1 deletion dist/js/field.js

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions resources/js/components/DetailField.vue
Original file line number Diff line number Diff line change
Expand Up @@ -88,8 +88,8 @@ export default {
setInitialValue: function () {
this.value = {
latitude: this.field.value.latitude || 0,
longitude: this.field.value.longitude || 0,
latitude: this.field.value[this.field.latitude || "latitude"] || 0,
longitude: this.field.value[this.field.longitude || "longitude"] || 0,
};
},
},
Expand Down

0 comments on commit d4d5eb6

Please sign in to comment.