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

[9.0][MIG][survey_percent_question] #21

Merged
merged 12 commits into from May 19, 2017

Conversation

tarteo
Copy link
Member

@tarteo tarteo commented Nov 4, 2016

No description provided.

@tarteo tarteo mentioned this pull request Nov 4, 2016
3 tasks
@coveralls
Copy link

Coverage Status

Coverage decreased (-45.8%) to 54.167% when pulling 5e44a21 on tarteo:90_mig_survey_percent_question into 970ffc9 on OCA:9.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-45.8%) to 54.167% when pulling 5e44a21 on tarteo:90_mig_survey_percent_question into 970ffc9 on OCA:9.0.

@coveralls
Copy link

Coverage Status

Coverage decreased (-45.8%) to 54.167% when pulling 5e44a21 on tarteo:90_mig_survey_percent_question into 970ffc9 on OCA:9.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-45.8%) to 54.167% when pulling 5e44a21 on tarteo:90_mig_survey_percent_question into 970ffc9 on OCA:9.0.

@tarteo
Copy link
Member Author

tarteo commented Nov 4, 2016

Oh, let me check the unit tests.

@coveralls
Copy link

coveralls commented Nov 7, 2016

Coverage Status

Coverage decreased (-29.2%) to 70.833% when pulling 042feb6 on tarteo:90_mig_survey_percent_question into 970ffc9 on OCA:9.0.

@coveralls
Copy link

coveralls commented Nov 8, 2016

Coverage Status

Coverage decreased (-2.08%) to 97.917% when pulling 922a159 on tarteo:90_mig_survey_percent_question into 970ffc9 on OCA:9.0.

@coveralls
Copy link

coveralls commented Nov 9, 2016

Coverage Status

Coverage decreased (-1.04%) to 98.958% when pulling 59bd720 on tarteo:90_mig_survey_percent_question into 970ffc9 on OCA:9.0.

@coveralls
Copy link

coveralls commented Nov 9, 2016

Coverage Status

Coverage decreased (-1.04%) to 98.958% when pulling 210f0cd on tarteo:90_mig_survey_percent_question into 970ffc9 on OCA:9.0.

@coveralls
Copy link

coveralls commented Nov 9, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling f10aeeb on tarteo:90_mig_survey_percent_question into 970ffc9 on OCA:9.0.

@tarteo
Copy link
Member Author

tarteo commented Nov 9, 2016

Unit tests are done!

#, python-format
msgid "{0} is not a number"
msgstr ""

Copy link
Member

Choose a reason for hiding this comment

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

.pot file can be removed, is not used anymore

@@ -0,0 +1,66 @@
.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg
:alt: License: AGPL-3
Copy link
Member

Choose a reason for hiding this comment

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

Please add the target attribute for the image, as per the readme template

===========

Bugs are tracked on `GitHub Issues
<https://github.com/OCA/200/issues>`_. In case of trouble, please
Copy link
Member

Choose a reason for hiding this comment

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

Link here must be:
https://github.com/OCA/survey/issues

@coveralls
Copy link

coveralls commented Mar 16, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 5b0ca6d on tarteo:90_mig_survey_percent_question into 970ffc9 on OCA:9.0.

@tarteo
Copy link
Member Author

tarteo commented Mar 20, 2017

@astirpe I fixed the README

Copy link
Member

@pankk pankk left a comment

Choose a reason for hiding this comment

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

Tested locally. I successfully created and viewed the results of a survey with percent questions.

@pedrobaeza pedrobaeza merged commit 60f50e4 into OCA:9.0 May 19, 2017
@tarteo tarteo deleted the 90_mig_survey_percent_question branch October 30, 2019 14:44
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.

None yet

5 participants