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
implement selection of location in portlet #23
Conversation
|
||
|
||
class IWeatherPortlet(IPortletDataProvider): | ||
"""A weather portlet. | ||
""" | ||
'''A weather portlet. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
according to PEP 257 -- Docstring Conventions:
For consistency, always use """triple double quotes""" around docstrings
so, please do not replace """ with '''.
Conflicts: src/collective/weather/portlets/weather.pt
Changes Unknown when pulling 7ee1ee0 on issue_15 into * on master*. |
Changes Unknown when pulling d89c3e3 on issue_15 into * on master*. |
IVocabularyFactory, name='collective.weather.Locations') | ||
vocab = factory(self.context) | ||
self.current_city = None | ||
if self.data.location in vocab: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what happens if the location is not in the vocabulary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
weather_info is not calculated and the portlet is not shown
Changes Unknown when pulling 2b5ddae on issue_15 into * on master*. |
Changes Unknown when pulling 2b5ddae on issue_15 into * on master*. |
Changes Unknown when pulling bff2f06 on issue_15 into * on master*. |
implement selection of location in portlet
fixes #15