Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add polling station intent #33

Closed
wants to merge 3 commits into from

Conversation

jaumb
Copy link
Collaborator

@jaumb jaumb commented Jun 12, 2019

closes #17

@jaumb jaumb requested review from jmartini, curranjm and Joshhw June 12, 2019 01:16
@jaumb
Copy link
Collaborator Author

jaumb commented Jul 17, 2019

@jmartini @hamstap85 ready for review when you have a chance

if current_address is None:
# Delegate to the Alexa interaction model for getting the user address
LOGGER.debug('Requesting user address.')
response.dialog_directive = "Delegate"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could alternatively use auto delegation

"""
Creates output speech for polling locations near the provided address

:param address: Current address
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing :param mycity_request

@jaumb jaumb force-pushed the 17_pollingStations branch from 1882a0d to 856d286 Compare July 17, 2019 00:42
@Joshhw
Copy link
Collaborator

Joshhw commented Apr 29, 2020

Closing since we already merged. Congrats @jaumb

@Joshhw Joshhw closed this Apr 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Respond to questions about polling locations
3 participants