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

[MERGE] survey, website_slides: implement YAB2B #36557

Closed

Conversation

@mcm-odoo
Copy link

commented Sep 9, 2019

PURPOSE

Implement Yet Another Back2Backs, coming from FP feedback.

SPECIFCIATIONS

Improve eLearning backend
Rework survey menus
Add some demo data for certification
Clean some code bits

Task 2066646

@robodoo robodoo added the seen 🙂 label Sep 9, 2019

@C3POdoo C3POdoo added the RD label Sep 9, 2019

@mcm-odoo mcm-odoo force-pushed the odoo-dev:saas-12.5-elearning-fp-feedback-mcm branch from e7da820 to 16d2116 Sep 9, 2019

@robodoo robodoo added the CI 🤖 label Sep 9, 2019

@mcm-odoo mcm-odoo force-pushed the odoo-dev:saas-12.5-elearning-fp-feedback-mcm branch from 16d2116 to e9d13f8 Sep 10, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Sep 10, 2019

mcm-odoo added 4 commits Sep 6, 2019
[IMP] website_slides: improve slides backend views
This commit improves a bit the existing backend views of the elearning
app (notably propagating content and course labeling instead of slide and
channel) and adds a pivot view for model slide.slide.

FP feedback

Task 2066646
[IMP] survey: improve menu items
This commit removes the configuration menu in survey app. Menu items are
distributed in the other menus: questions, answers.

FP feedback

Task 2066646
[IMP] (website_slides_)survey: add demo data for certifications
This commit adds a few demo data for the certifications.

FP feedback

Task 2066646
[IMP] survey: little optimization for user_input_line creation/update
Before:
  The method _get_mark called Model.exists() just after calling
  Model.browse
  This process makes 2 calls to database to do almost the same thing

Now:
  Removed the method _get_mark and directly call Model.browse

FP feedback

Task 2066646

@tde-banana-odoo tde-banana-odoo force-pushed the odoo-dev:saas-12.5-elearning-fp-feedback-mcm branch from e9d13f8 to d6dfe2e Sep 11, 2019

@robodoo robodoo removed the CI 🤖 label Sep 11, 2019

@tde-banana-odoo tde-banana-odoo changed the title Saas 12.5 elearning fp feedback mcm [MERGE] survey, website_slides: YAB2B Sep 11, 2019

@tde-banana-odoo tde-banana-odoo changed the title [MERGE] survey, website_slides: YAB2B [MERGE] survey, website_slides: implement YAB2B Sep 11, 2019

@tde-banana-odoo

This comment has been minimized.

Copy link
Contributor

commented Sep 11, 2019

@robodoo r+ rebase-merge

@robodoo robodoo added the r+ 👌 label Sep 11, 2019

@robodoo

This comment has been minimized.

Copy link
Contributor

commented Sep 11, 2019

Merge method set to rebase and merge, using the PR as merge commit message

@robodoo robodoo added the CI 🤖 label Sep 11, 2019

robodoo added a commit that referenced this pull request Sep 11, 2019
[MERGE] survey, website_slides: implement YAB2B
PURPOSE

Implement Yet Another Back2Backs, coming from FP feedback.

SPECIFCIATIONS

Improve eLearning backend
Rework survey menus
Add some demo data for certification
Clean some code bits

Task 2066646

closes #36557

Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
@robodoo

This comment has been minimized.

Copy link
Contributor

commented Sep 11, 2019

Merged at 6f8cace, thanks!

@robodoo robodoo closed this Sep 11, 2019

@tde-banana-odoo tde-banana-odoo deleted the odoo-dev:saas-12.5-elearning-fp-feedback-mcm branch Sep 12, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.