Skip to content
Snippets Groups Projects
  1. Aug 05, 2019
  2. Jul 22, 2019
  3. Aug 05, 2019
    • Kevin Baptiste's avatar
      [FIX] base_setup: remove mobile apps icon during search · c5ab50cc
      Kevin Baptiste authored
      
      The mobile apps icons were always showing even during a search in the
      settings.
      
      closes odoo/odoo#35325
      
      Signed-off-by: default avatarRomain Libert (rli) <rli@odoo.com>
      c5ab50cc
    • Robot Odoo's avatar
      [MERGE][REF] website_slides: make eLearning an application with its own backend · fb2522ac
      Robot Odoo authored
      PURPOSE
      
      eLearning should have its own application in manager. Indeed this is becoming
      a big application and having it embedded inside Website application is not
      enough anymore. It should also hold updated and easy-to-use menus,
      navigation, actions and views in backend.
      
      SPECIFICATIONS
      
      Update manifest and make website_slides an application separated from
      website embedded menus.
      
      Implement new menu structure
      
      * Courses
      * Courses
      List view: Course Name, # Views, # Attendees, # Ratings, Revenues,
      Reviews
      * Contents
      * Reviews: kanban view of channel reviews
      * Certifications: kanban view of surveys of type certificate
      
      * Forum
      * Forums: add columns to list view: Posts (questions count), Views
      (sum of views of each post), Answers (sum of child_count on each post),
      Favorites (sum of favourite_count on each post)
      * Posts: add columns to list view: Views / Answers / Favourites
      
      * Reporting
      * Courses: list view on slide.channel model, ordered by # views.
      List view: Course Name, # Views, # Attendees, # Ratings, Revenues;
      * Contents: graph view on slide.slide model
      Area Graph: X month
      Y measure: # Views (others measures: Attendees, ...)
      Group by: course (color)
      * Revenues: graph view on sale.report model showing revenues of eLearning
      courses.
      Area chart, X: monthn Y: €, Color: course
      * Certifications: list view on surveys of type certificate
      Columns: Title, Survey Stage, Registered People, Certified People, Attempts
      Success Rate (certified/attempts), Average Results
      * Reviews: see kanban of latest reviews;
      * Quiz: list view on slide.question model.
      List view: Add # Quizz Trials, # Quizz Passed, Avg Attempts
      * Forum: graph view on forum.post model
      Area chart: # Posts created by months, color=forum
      
      * Configuration
      * Settings
      * Allow Forum on Courses (installs website_slides_forum)
      * Allow Certifications (installs website_slides_survey)
      * Allow Selling Courses (installs website_sale_slides)
      * Course Tags
      * See channel.tags.group with label Course Tags, form view allow to
      configure group tags
      * https://drive.google.com/a/odoo.com/file/d/1G0g2UnG5cZG55EXQ5icbgwe7T2eSZskT/view?usp=drivesdk
      * On click: open :https://drive.google.com/a/odoo.com/file/d/1I_PW3aZAbrslXadiYqmVF-_6dGXx8knd/view?usp=drivesdk
      * Content Tags
      
      Add course (slide.channel) kanban view. Each tile should contain
      
      * Title of the course
      * Tags below the title
      * Primary button "New Lesson"
      * Statistics :
      * Attendees
      * Running (progress != 100%)
      * (if one content is certificate) Certified or Finished
      * See : https://drive.google.com/a/odoo.com/file/d/13K8FmkbwmBG4JQ5gH3Q5T8OlWPs3blcu/view?usp=drivesdk
      * With FP remarks https://drive.google.com/a/odoo.com/file/d/1mb7LUT0VUNs5fMcKB9c0Bfaqk2-XFFuu/view?usp=drivesdk
      * Add "Sales       150.00$" below "Watch Time" if type "on payment". Same
      info than the smart/stat button on the course itself. Click on the amount
      sends to Reporting Revenues (same than on the stat btn)
      
      Add review (rating.rating) kanban view. Each tile should contain
      
      * Kanban view, full width, no default group by
      * Order by date (most recent on top)
      * See: https://drive.google.com/a/odoo.com/file/d/1IYE1QBIJa26BLugRiafyBop-srkYwndC/view?usp=drivesdk
      * add a tag for unpublished comments
      * Add the pic of the user above the stars
      * Click on card, send to front-end and able to (un)publish the comment
      * Kind of what's on the front-end:  https://drive.g
      
      
      
      Improve course form view: add a revenue stat button displaying the total
      of sale order linked to the course's product.
      
      Integrate a button on slide.channel form view allowing to perform a mass
      mailing on course attendees. Do a flow similar to event.
      
      Remove "Our Documentation" dummy default existing data. It makes no sense
      to have void data. Instead just add a small explanation text in eLearning
      front-end telling the user to create its first course if he has rights to
      do so.
      
      In order to better show the new backend add some demo data for missing
      numbers, notably
      
      * sales: add confirmed sale orders in order to have revenues value;
      * forum: add some posts and answers;
      * quiz: update slide.slide.partner demo data to hold quiz attempts count
      
      LINKS
      
      Task 1978729
      PR #35061
      
      Co-Authored-By: default avatarThibault Delavallée <tde@odoo.com>
      Co-Authored-By: default avatarJérémy Hennecart <jeh@odoo.com>
      
      Signed-off-by: default avatarThibault Delavallee (tde) <tde@openerp.com>
      Unverified
      fb2522ac
    • Thibault Delavallée's avatar
      [REV] website_sale: set Cart recovery back in form view · 3d737bb3
      Thibault Delavallée authored
      Partially reverts d781d6f6ea504dcc89f412b94774df6bd08c78aa
      
      Indeed modified tour has to go through various list view actions and is
      likely to crash because of some complicated triggers to write. Let us have
      the recovery action back to form view. Anyway it makes no harm to have it
      available in action menu.
      
      LINKS
      
      Task 1978729
      PR #35061
      3d737bb3
    • Jérémy Hennecart's avatar
      [IMP] website_slides(_*): improve demo data and remove dummy data · aeb23b21
      Jérémy Hennecart authored
      
      PURPOSE
      
      eLearning should have its own application in manager. Indeed this is becoming
      a big application and having it embedded inside Website application is not
      enough anymore. It should also hold updated and easy-to-use menus,
      navigation, actions and views in backend.
      
      SPECIFICATIONS
      
      Remove "Our Documentation" dummy default existing data. It makes no sense
      to have void data. Instead just add a small explanation text in eLearning
      front-end telling the user to create its first course if he has rights to
      do so.
      
      In order to better show the new backend add some demo data for missing
      numbers, notably
      
        * sales: add confirmed sale orders in order to have revenues value;
        * forum: add some posts and answers;
        * quiz: update slide.slide.partner demo data to hold quiz attempts count;
      
      LINKS
      
      Task 1978729
      PR #35061
      
      Co-Authored-By: default avatarThibault Delavallée <tde@odoo.com>
      Co-Authored-By: default avatarJérémy Hennecart <jeh@odoo.com>
      aeb23b21
    • Jérémy Hennecart's avatar
      [REF] website_slide_survey: improve integration into eLeaning application · ee29641b
      Jérémy Hennecart authored
      
      PURPOSE
      
      eLearning should have its own application in manager. Indeed this is becoming
      a big application and having it embedded inside Website application is not
      enough anymore. It should also hold updated and easy-to-use menus,
      navigation, actions and views in backend.
      
      SPECIFICATIONS
      
      Add menu entry inside Courses
        * Certifications: kanban view of surveys of type certificate
      
      Add menu entry inside Reporting
        * Certifications: list view on surveys of type certificate
          Columns: Title, Survey Stage, Registered People, Certified People, Attempts
          Success Rate (certified/attempts), Average Results
      
      LINKS
      
      Task 1978729
      PR #35061
      
      Co-Authored-By: default avatarThibault Delavallée <tde@odoo.com>
      Co-Authored-By: default avatarJérémy Hennecart <jeh@odoo.com>
      ee29641b
    • Jérémy Hennecart's avatar
      [REF] website_sale_slides: improve integration into eLeaning application · ee10b19f
      Jérémy Hennecart authored
      
      PURPOSE
      
      eLearning should have its own application in manager. Indeed this is becoming
      a big application and having it embedded inside Website application is not
      enough anymore. It should also hold updated and easy-to-use menus,
      navigation, actions and views in backend.
      
      SPECIFICATIONS
      
      Improve course form view: add a revenue stat button displaying the total
      of sale order linked to the course's product.
      
      Add in slide.channel kanban card
        * Add "Sales       150.00$" below "Watch Time" if type "on payment". Same
          info than the smart/stat button on the course itself. Click on the amount
          sends to Reporting Revenues (same than on the stat btn)
      
      Add menu entry inside Reporting
          * Revenues: graph view on sale.report model showing revenues of eLearning
            courses.
              Area chart, X: month Y: €, Color: course
      
      LINKS
      
      Task 1978729
      PR #35061
      
      Co-Authored-By: default avatarThibault Delavallée <tde@odoo.com>
      Co-Authored-By: default avatarJérémy Hennecart <jeh@odoo.com>
      ee10b19f
    • Jérémy Hennecart's avatar
      [REF] website_slides_forum: improve integration into eLeaning application · 2e707935
      Jérémy Hennecart authored
      
      PURPOSE
      
      eLearning should have its own application in manager. Indeed this is becoming
      a big application and having it embedded inside Website application is not
      enough anymore. It should also hold updated and easy-to-use menus,
      navigation, actions and views in backend.
      
      SPECIFICATIONS
      
      Add menu after Courses
      
        * Forum
          * Forums: add columns to list view: Posts (questions count), Views
            (sum of views of each post), Answers (sum of child_count on each post),
            Favorites (sum of favourite_count on each post)
          * Posts: add columns to list view: Views / Answers / Favourites
      
      Add menu entry inside Reporting
        * Forum: graph view on forum.post model
            Area chart: # Posts created by months, color=forum
      
      LINKS
      
      Task 1978729
      PR #35061
      
      Co-Authored-By: default avatarThibault Delavallée <tde@odoo.com>
      Co-Authored-By: default avatarJérémy Hennecart <jeh@odoo.com>
      2e707935
    • Jérémy Hennecart's avatar
      [ADD] mass_mailing_slides: allow to mass mail course attendees · 313a7d9e
      Jérémy Hennecart authored
      
      PURPOSE
      
      eLearning should have its own application in manager. Indeed this is becoming
      a big application and having it embedded inside Website application is not
      enough anymore. It should also hold updated and easy-to-use menus,
      navigation, actions and views in backend.
      
      SPECIFICATIONS
      
      Integrate a button on slide.channel form view allowing to perform a mass
      mailing on course attendees. Do a flow similar to event.
      
      Add menu entry in menu of slide.channel kanban card after invite
        * Mail Attendees (same as on the course card and depend of the setting)
      
      Add menu entry inside Settings
        * Settings: add an option to install module mass_mailing_slide
      
      LINKS
      
      Task 1978729
      PR #35061
      
      Co-Authored-By: default avatarThibault Delavallée <tde@odoo.com>
      Co-Authored-By: default avatarJérémy Hennecart <jeh@odoo.com>
      313a7d9e
    • Thibault Delavallée's avatar
      [FIX] mass_mailing: correctly check mailing domain · fd9962cd
      Thibault Delavallée authored
      Currently each change in mailing model resets the mailing domain. Idea behind
      that behavior is that domain for a given model will probably not be valid for
      another model because of fields being model specific.
      
      However this breaks when creating mailing with default values for mailing
      model. Indeed even valid domains are reset to a default void / opt-out-ed
      value due to current onchange implementation.
      
      This commit fixes that behavior by resetting only invalid domain i.e.
      that crashes when performing a search on target model.
      
      LINKS
      
      Task 1978729
      PR #35061
      fd9962cd
    • Thibault Delavallée's avatar
      [IMP] website_slides: add an overview (kanban view) on course model · 98bdb241
      Thibault Delavallée authored
      PURPOSE
      
      eLearning should have its own application in manager. Indeed this is becoming
      a big application and having it embedded inside Website application is not
      enough anymore. It should also hold updated and easy-to-use menus,
      navigation, actions and views in backend.
      
      SPECIFICATIONS
      
      Add course (slide.channel) kanban view. Each tile should contain
      
        * Title of the course (make it big, o_primary)
        * Tags below the title
        * Primary button "New Lesson"
        * Statistics :
          * Attendees
          * Running (progress != 100%)
          * (if one content is certificate) Certified or Finished
        * See
          * v1: https://drive.google.com/a/odoo.com/file/d/13K8FmkbwmBG4JQ5gH3Q5T8OlWPs3blcu/view?usp=drivesdk
      
      With FP remarks https://drive.google.com/a/odoo.com/file/d/1mb7LUT0VUNs5fMcKB9c0Bfaqk2-XFFuu/view?usp=drivesdk
      
      
        * remove globe, type and visibility of course, lessen padding between rating
          views and watch time;
        * for rating, instead of 8.00 => 5 reviews (4.5/5);
        * replace "View Lesson" / "Invite" by "View Course" => Send to the front-end
        * In the menu of the card, add:
          * Invite (same behavior than before, when it was a primary btn)
        * click on x Contents:  land on the contents, group by category with sequence
        * re-order of the 3 infos/Stats at the bottom of the card: 1. Content,
          2. Attendee, 3. Certified
          * + number: font-weight: 500 + font-size 1.2em
          * + labels: remove text-uppercase, + text-color: #666666
          * + the "whole box" is clickable
      
      LINKS
      
      Task 1978729
      PR #35061
      
      Co-Authored-By: default avatarThibault Delavallée <tde@odoo.com>
      Co-Authored-By: default avatarJérémy Hennecart <jeh@odoo.com>
      98bdb241
    • Thibault Delavallée's avatar
      [IMP] website_slides: add a new kanban view for contents (slide.slide) · 665d0dfa
      Thibault Delavallée authored
      PURPOSE
      
      eLearning should have its own application in manager. Indeed this is becoming
      a big application and having it embedded inside Website application is not
      enough anymore. It should also hold updated and easy-to-use menus,
      navigation, actions and views in backend.
      
      SPECIFICATIONS
      
      Add a kanban view for slide.slide model
      
        * See https://drive.google.com/a/odoo.com/file/d/1WKj13-C_UCOwh4dN-FRmU9Pp6BsZ_HvT/view?usp=drivesdk
      
      LINKS
      
      Task 1978729
      PR #35061
      665d0dfa
    • Thibault Delavallée's avatar
      [IMP] website_slides: add a new kanban view for ratings · 54192e46
      Thibault Delavallée authored
      PURPOSE
      
      eLearning should have its own application in manager. Indeed this is becoming
      a big application and having it embedded inside Website application is not
      enough anymore. It should also hold updated and easy-to-use menus,
      navigation, actions and views in backend.
      
      SPECIFICATIONS
      
      Add in menu Courses, a new entry Reviews
        * Kanban view, full width, no default group by
          * Order by date (most recent on top)
          * See: https://drive.google.com/a/odoo.com/file/d/1IYE1QBIJa26BLugRiafyBop-srkYwndC/view?usp=drivesdk
          * add a tag for unpublished comments
          * Add the pic of the user above the stars
          * Click on card, send to front-end and able to (un)publish the comment
          * Kind of what's on the front-end:  https://drive.google.com/a/odoo.com/file/d/1e9aQSWvUM2QOynRT08XSUzIjFiGnjsv8/view?usp=drivesdk
      
      LINKS
      
      Task 1978729
      PR #35061
      54192e46
    • Jérémy Hennecart's avatar
      [REF] website_slides: make eLearning an application with its own menu structure · 4587e6a3
      Jérémy Hennecart authored
      PURPOSE
      
      eLearning should have its own application in manager. Indeed this is becoming
      a big application and having it embedded inside Website application is not
      enough anymore. It should also hold updated and easy-to-use menus,
      navigation, actions and views in backend.
      
      SPECIFICATIONS
      
      Update manifest and make website_slides an application separated from
      website embedded menus.
      
      Implement new menu structure
      
        * Courses
          * Courses
            List view: Course Name, # Views, # Attendees, # Ratings, Revenues,
            Reviews
          * Contents
      
        * Reporting
          * Courses: list view on slide.channel model, ordered by # views.
              * add a graph view (group by content)
          * Contents: graph view on slide.slide model
              Area Graph: X month
              Y measure: # Views (others measures: Attendees, ...)
              Group by: course (color)
          * Reviews: see kanban of latest reviews;
          * Quiz: list view on slide.question model.
              List view: Add # Quizz Trials, # Quizz Passed, Avg Attempts
      
        * Configuration
          * Settings
            * Allow Forum on Courses (installs website_slides_forum)
            * Allow Certifications (installs website_slides_survey)
            * Allow Selling Courses (installs website_sale_slides)
          * Course Tags
            * See channel.tags.group with label Course Tags, form view allow to
              configure group tags
            * https://drive.google.com/a/odoo.com/file/d/1G0g2UnG5cZG55EXQ5icbgwe7T2eSZskT/view?usp=drivesdk
            * On click: open :https://drive.google.com/a/odoo.com/file/d/1I_PW3aZAbrslXadiYqmVF-_6dGXx8knd/view?usp=drivesdk
      
      
          * Content Tags
      
      This structure will gain new items in sub-modules, notably bridge with
      form and website_sale. Those will be implemented in next commits.
      
      LINKS
      
      Task 1978729
      PR #35061
      
      Co-Authored-By: default avatarThibault Delavallée <tde@odoo.com>
      Co-Authored-By: default avatarJérémy Hennecart <jeh@odoo.com>
      4587e6a3
    • Thibault Delavallée's avatar
      [FIX] website_slides: fix breadcrumb when viewing a slide from a category search · ad9d7c5c
      Thibault Delavallée authored
      Template expects a category record but controller gives the ID of it.
      Oversight of 4ccd859b
      
      LINKS
      
      Task 1978729
      PR #35061
      ad9d7c5c
  4. Aug 02, 2019
  5. Jul 30, 2019
  6. Aug 03, 2019
    • Robot Odoo's avatar
      [IMP] *: correctly read/write website_published · 9885ccfd
      Robot Odoo authored
      
      The `website_published` field from the website's mixins is basically a readonly
      from `is_published` field.
      On read, this field will simply read `is_published` and check if the record's
      website_id is accessible (only for the multi mixin).
      On write, it will always write on `is_published`.
      
      Whenever possible, any access (read/search) on published state should be done on `website_published`.
      Any write should be done on `is_published`.
      
      This PR improves a few things:
      - A lot of code was writting on website_published which was just then writting
      on is_published. Writting directly on is_published makes more sense.
      - Some backend fields would still reference `website_published` instead of
      `is_published` which would just go through the related for no reason.
      Plus, using `is_published` will make the field tooltip more accurate as we
      are not in a website context ('Visible on current website' to 'Is Published')
      - Filter and search on tree view were still using the `website_published`
      related field, which is just a readonly when we are not in a frontend
      context.
      - Some create and write function would have security check on
      `website_published` value but that was wrong as the user could bypass that by
      simply writting on `is_published`. For the write method, check `is_published`
      is more accurate as it will cover both case since `website_published` will
      then call the write method on `is_published`
      - Some models would create their own `website_published` field rather than using the
      `website.published.mixin` mixin which should be used instead.
      That was done to be able to override the default value of the boolean.
      This commit simply adds the possibility to override the default value of the
      mixin field.
      - Fix wrong publish security check that were only checking `website_published` in vals.
      
      closes odoo/odoo#35189
      
      Signed-off-by: default avatarJérémy Kersten (jke) <jke@openerp.com>
      Unverified
      9885ccfd
    • Romain Derie's avatar
      [FIX] website: correctly check security of publish mixin · 4086f344
      Romain Derie authored
      Commit 91696805 introduced a mechanism to check rights when writing or
      creating on `website_published` field.
      Problem is, `website_published` is just kind of a related to `is_published`
      which really stores the value.
      
      Thus, the checks would not work when writing/creating on `is_published`.
      That would leave a loophole to bypass the expected behavior.
      4086f344
    • Romain Derie's avatar
      [IMP] website*: use published mixin rather than own field · 7a90dedd
      Romain Derie authored
      *website_crm_partner_assign, website_customer
      
      Before this commit, some models would create their own `website_published`
      field rather than using the `website.published.mixin` mixin which should be
      used instead.
      
      That was done to be able to override the default value of the boolean.
      
      This commit simply adds the possibility to override the default value of the
      mixin field.
      
      Also, help and string could simply be added on xml views rather than overriding
      the field.
      7a90dedd
    • Romain Derie's avatar
      [IMP] *: correctly read/write website_published · cc1d78a8
      Romain Derie authored
      The `website_published` field from the website's mixins is basically a readonly
      from `is_published` field.
      On read, this field will simply read `is_published` and check if the record's
      website_id is accessible (only for the multi mixin).
      On write, it will always write on `is_published`.
      
      This commit improves a few things:
      - A lot of code was writting on website_published which was just then writting
        on is_published. Writting directly on is_published makes more sense.
      - Some backend fields would still reference `website_published` instead of
        `is_published` which would just go through the related for no reason.
        Plus, using `is_published` will make the field tooltip more accurate as we
        are not in a website context ('Visible on current website' to 'Is Published')
      - Filter and search on tree view were still using the `website_published`
        related field, which is just a readonly when we are not in a frontend
        context.
      - Some create and write function would have security check on
        `website_published` value but that was wrong as the user could bypass that by
        simply writting on `is_published`. For the write method, check `is_published`
        is more accurate as it will cover both case since `website_published` will
        then call the write method on `is_published`
      cc1d78a8
  7. Aug 02, 2019
  8. Jul 24, 2019
  9. Aug 02, 2019
Loading