Skip to content
This repository has been archived by the owner on Sep 8, 2024. It is now read-only.

Integrate location with backend #487

Closed
augustnmonteiro opened this issue Jan 31, 2017 · 0 comments
Closed

Integrate location with backend #487

augustnmonteiro opened this issue Jan 31, 2017 · 0 comments
Assignees
@augustnmonteiro

Comments

@augustnmonteiro
Copy link
Contributor

  • Configuration should get the location of device and save on mycroft.conf
@augustnmonteiro augustnmonteiro self-assigned this Jan 31, 2017
augustnmonteiro added a commit that referenced this issue Jan 31, 2017
@augustnmonteiro
#487 - Integrating location microservice with mycroft core
de7281c
augustnmonteiro added a commit that referenced this issue Jan 31, 2017
@augustnmonteiro
#487 - Saving all location object in the configuration instead of onl…
71726b5 
…y the name
augustnmonteiro added a commit that referenced this issue Jan 31, 2017
@augustnmonteiro
#487 - Adding default values for location on weather skill
9ae41a0
augustnmonteiro added a commit that referenced this issue Jan 31, 2017
@augustnmonteiro
#487 - Checking if location exists if doesn't say location not found …
db57659 
…dialog
augustnmonteiro added a commit that referenced this issue Feb 1, 2017
@augustnmonteiro
#487 - Integrate Location with mycroft-core
e310789 
- Adding more phrases on location not found dialog
- Creating get_location method
- Fixing identation
augustnmonteiro added a commit that referenced this issue Feb 1, 2017
@augustnmonteiro
#487 - Updating synchronize configuration time
268f7fa
augustnmonteiro added a commit that referenced this issue Feb 2, 2017
@augustnmonteiro
#487 - Integrating location microservice with mycroft core
540cd89
augustnmonteiro added a commit that referenced this issue Feb 2, 2017
@augustnmonteiro
#487 - Saving all location object in the configuration instead of onl…
bed6175 
…y the name
augustnmonteiro added a commit that referenced this issue Feb 2, 2017
@augustnmonteiro
#487 - Adding default values for location on weather skill
d1068ab
augustnmonteiro added a commit that referenced this issue Feb 2, 2017
@augustnmonteiro
#487 - Checking if location exists if doesn't say location not found …
3e09255 
…dialog
augustnmonteiro added a commit that referenced this issue Feb 2, 2017
@augustnmonteiro
#487 - Integrate Location with mycroft-core
c07cdae 
- Adding more phrases on location not found dialog
- Creating get_location method
- Fixing identation
augustnmonteiro added a commit that referenced this issue Feb 2, 2017
@augustnmonteiro
#487 - Updating synchronize configuration time
fc71c9f
penrods pushed a commit that referenced this issue Feb 3, 2017
More Location integration for #487
c6d20b8 
For the Weather Skill:
* When talking about the current city, the city name is generally not spoken (more natural)
* A "pretty" name of just the city is used instead of the complete name
* Works around the recurring issue with OWM where they report bad min/max temps (same as the current temp)
* Changed "Location is not valid" to "I don't know that location" (people don't say "not valid")

For the Time Skill:
* The timezone is extracted from the device location setting
* Time responses are more varied and shorter

This change adds MycroftSkill.location_pretty and MycroftSkill.location_timezone properties.
aatchison pushed a commit that referenced this issue Feb 3, 2017
@aatchison
More Location integration for #487
acfdff4 
For the Weather Skill:
* When talking about the current city, the city name is generally not spoken (more natural)
* A "pretty" name of just the city is used instead of the complete name
* Works around the recurring issue with OWM where they report bad min/max temps (same as the current temp)
* Changed "Location is not valid" to "I don't know that location" (people don't say "not valid")

For the Time Skill:
* The timezone is extracted from the device location setting
* Time responses are more varied and shorter

This change adds MycroftSkill.location_pretty and MycroftSkill.location_timezone properties.
forslund pushed a commit to forslund/mycroft-core that referenced this issue Feb 9, 2017
@augustnmonteiro @forslund
MycroftAI#487 - Integrating location microservice with mycroft core
91e7aa9
forslund pushed a commit to forslund/mycroft-core that referenced this issue Feb 9, 2017
@augustnmonteiro @forslund
MycroftAI#487 - Saving all location object in the configuration inste…
7c94ac6 
…ad of only the name
forslund pushed a commit to forslund/mycroft-core that referenced this issue Feb 9, 2017
@augustnmonteiro @forslund
MycroftAI#487 - Adding default values for location on weather skill
b1b2d65
forslund pushed a commit to forslund/mycroft-core that referenced this issue Feb 9, 2017
@augustnmonteiro @forslund
MycroftAI#487 - Checking if location exists if doesn't say location n…
69c7b6e 
…ot found dialog
forslund pushed a commit to forslund/mycroft-core that referenced this issue Feb 9, 2017
@augustnmonteiro @forslund
MycroftAI#487 - Integrate Location with mycroft-core
602ddb6 
- Adding more phrases on location not found dialog
- Creating get_location method
- Fixing identation
forslund pushed a commit to forslund/mycroft-core that referenced this issue Feb 9, 2017
@augustnmonteiro @forslund
MycroftAI#487 - Updating synchronize configuration time
addf1e9
forslund pushed a commit to forslund/mycroft-core that referenced this issue Feb 9, 2017
@forslund
More Location integration for MycroftAI#487
f8e7000 
For the Weather Skill:
* When talking about the current city, the city name is generally not spoken (more natural)
* A "pretty" name of just the city is used instead of the complete name
* Works around the recurring issue with OWM where they report bad min/max temps (same as the current temp)
* Changed "Location is not valid" to "I don't know that location" (people don't say "not valid")

For the Time Skill:
* The timezone is extracted from the device location setting
* Time responses are more varied and shorter

This change adds MycroftSkill.location_pretty and MycroftSkill.location_timezone properties.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@augustnmonteiro augustnmonteiro

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@augustnmonteiro @aatchison