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

[MERGE][REF] survey*_: improve survey demo data #29303

Merged
merged 3 commits into from Dec 6, 2018

Conversation

Projects
None yet
3 participants
@tde-banana-odoo
Contributor

tde-banana-odoo commented Dec 6, 2018

Purpose of this merge is to update the demo data for surveys used in various
applications, notably the base one in survey.

It impacts survey (feedback survey to make coherent with odoo demo data and
remove references to odoo as a product in odoo), hr_recruitment_survey (sample
recruitment survey moved from data to demo and simplified).

This merge is linked to task ID 1912228 and PR #29303.

@robodoo robodoo added the seen 🙂 label Dec 6, 2018

@C3POdoo C3POdoo added the RD label Dec 6, 2018

@tde-banana-odoo tde-banana-odoo force-pushed the odoo-dev:master-survey-demo-tde branch from 2a300fb to e9ba627 Dec 6, 2018

tde-banana-odoo added a commit to odoo-dev/odoo that referenced this pull request Dec 6, 2018

[MOV] hr_recruitment_survey: set demo survey as demo instead of data
It is absolutely not necessary to have a small recruitment survey as a
data. It should be a demo, notably because it is used in demo jobs to give
a sample survey.

This commit is linked to task ID 1912228 and PR odoo#29303.

tde-banana-odoo added a commit to odoo-dev/odoo that referenced this pull request Dec 6, 2018

[IMP] hr_recruitment_survey: clean survey demo
Purpose of this merge is to simplify demo survey in hr_recruitment_survey,
make it shorted, simpler and a bit more updated.

This commit is linked to task ID 1912228 and PR odoo#29303.

tde-banana-odoo added some commits Dec 5, 2018

[MOV] hr_recruitment_survey: set demo survey as demo instead of data
It is absolutely not necessary to have a small recruitment survey as a
data. It should be a demo, notably because it is used in demo jobs in
order to have a sample survey available.

This commit is linked to task ID 1912228 and PR #29303.
[IMP] hr_recruitment_survey: clean demo survey
Purpose of this merge is to simplify demo survey in hr_recruitment_survey,
make it shorter, simpler and a bit more updated.

This commit is linked to task ID 1912228 and PR #29303.
[IMP] survey: improve demo survey
Purpose of this commit is to provide a new simpler survey as demo data in
survey application. Current survey is about Odoo use which is not coherent
with the demo data set used currently in Odoo. It is now a survey about
ecommerce use with an example of each different questions.

Survey tour is updated accordingly. It now uses a standard routing instead
of the test route. Indeed purpose of tests is to use real life use case
not corner case like testing capabilities of survey application.

This commit is linked to task ID 1912228 and PR #29303.

@robodoo robodoo added the CI 🤖 label Dec 6, 2018

@tde-banana-odoo tde-banana-odoo force-pushed the odoo-dev:master-survey-demo-tde branch from e9ba627 to 7df9cd7 Dec 6, 2018

@robodoo robodoo removed the CI 🤖 label Dec 6, 2018

@tde-banana-odoo

This comment has been minimized.

Contributor

tde-banana-odoo commented Dec 6, 2018

@robodoo r+ rebase-merge

@tde-banana-odoo

This comment has been minimized.

Contributor

tde-banana-odoo commented Dec 6, 2018

@robodoo r+ rebase merge

@robodoo

This comment has been minimized.

Contributor

robodoo commented Dec 6, 2018

I'm sorry, @tde-banana-odoo. This PR is already reviewed, reviewing it again is useless.

robodoo added a commit that referenced this pull request Dec 6, 2018

[MERGE][REF] survey*_: improve survey demo data
Purpose of this merge is to update the demo data for surveys used in various
applications, notably the base one in survey.

It impacts survey (feedback survey to make coherent with odoo demo data and
remove references to odoo as a product in odoo), hr_recruitment_survey (sample
recruitment survey moved from data to demo and simplified).

This merge is linked to task ID 1912228 and PR #29303.
@robodoo

This comment has been minimized.

Contributor

robodoo commented Dec 6, 2018

Merged, thanks!

@robodoo robodoo closed this Dec 6, 2018

@robodoo robodoo merged commit 7df9cd7 into odoo:master Dec 6, 2018

2 checks passed

ci/runbot runbot build 410872-29303-7df9cd (runtime 1812s)
Details
legal/cla Thibault Delavallée Odoo CLA signature check
Details

@tde-banana-odoo tde-banana-odoo deleted the odoo-dev:master-survey-demo-tde branch Dec 7, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment