search

OCA / survey

Survey addons
GNU Affero General Public License v3.0
27 stars 110 forks source link

[ADD] survey_percent_question #4

Closed veloutin closed 8 years ago

veloutin commented 9 years ago

This module adds a new question type "Percent" which allows splitting a percentage value across a list of options.

Ehtaga commented 9 years ago

Looks good to me :+1:

pedrobaeza commented 9 years ago

You should split the module in 2: one for the backend, and another for the website part.

rafaelbn commented 8 years ago

Please @JavierIniesta @JMODaERP review, thanks!

rafaelbn commented 8 years ago

You should split the module in 2: one for the backend, and another for the website part.

Hi @veloutin @Ehtaga did you split module as @pedrobaeza said?

Thanks a lot for you contributions and sorry in this big delay.

gurneyalex commented 8 years ago

@pedrobaeza what's the benefit of splitting?

pedrobaeza commented 8 years ago

Don't depend on frontend part if you only want percentage on backend surveys (for example, for internal HR evaluation)

rafaelbn commented 8 years ago

Hi @Ehtaga @veloutin I'm sorry about this PR. I would like to review it and go ahead if you are interested. Are you? @pedrobaeza survey module depends on website, so is not possible to work only with backend. I'm rebuilding runbot for testing

rafaelbn commented 8 years ago

Tested in runbot :+1:

rafaelbn commented 8 years ago

Ready to be merged