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

[ADD] learning_mangement_services: Learning Management Services #29

Open
wants to merge 21 commits into
base: 17.0
Choose a base branch
from

Conversation

khva-odoo
Copy link
Contributor

This module provides the infrastructure for delivering, managing, and tracking online courses.
They offer features such as course creation, student enrolment, assessments, progress tracking,
communication tools, certification and gamification.

task id: 3460428

learning_management_services/__manifest__.py Outdated Show resolved Hide resolved
learning_management_services/data/appointment_type.xml Outdated Show resolved Hide resolved
learning_management_services/data/loyalty_reward.xml Outdated Show resolved Hide resolved
learning_management_services/data/product_template.xml Outdated Show resolved Hide resolved
learning_management_services/demo/res_partner.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website_view.xml Outdated Show resolved Hide resolved
learning_management_services/__manifest__.py Outdated Show resolved Hide resolved
learning_management_services/__manifest__.py Outdated Show resolved Hide resolved
learning_management_services/data/loyalty_program.xml Outdated Show resolved Hide resolved
learning_management_services/data/product_pricelist.xml Outdated Show resolved Hide resolved
learning_management_services/data/slide_channel.xml Outdated Show resolved Hide resolved
learning_management_services/data/slide_slide.xml Outdated Show resolved Hide resolved
learning_management_services/data/survey_question.xml Outdated Show resolved Hide resolved
learning_management_services/data/survey_survey.xml Outdated Show resolved Hide resolved
learning_management_services/data/survey_survey.xml Outdated Show resolved Hide resolved
<field name="session_code">9404</field>
<field name="color">2</field>
<field name="users_can_go_back" eval="True" />
<field name="certification" eval="True" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<field name="certification" eval="True" />

compute field

learning_management_services/demo/forum_forum.xml Outdated Show resolved Hide resolved
learning_management_services/demo/forum_post.xml Outdated Show resolved Hide resolved
learning_management_services/demo/forum_post.xml Outdated Show resolved Hide resolved
learning_management_services/demo/loyalty_card.xml Outdated Show resolved Hide resolved
learning_management_services/demo/res_partner.xml Outdated Show resolved Hide resolved
learning_management_services/demo/res_partner.xml Outdated Show resolved Hide resolved
learning_management_services/demo/res_partner.xml Outdated Show resolved Hide resolved
learning_management_services/demo/res_partner.xml Outdated Show resolved Hide resolved
learning_management_services/demo/sale_order_confirm.xml Outdated Show resolved Hide resolved
@jva-odoo jva-odoo dismissed paso-odoo’s stale review November 7, 2023 18:32

it's already done and discussed

@jva-odoo jva-odoo added the ready PR is ready for review and integration label Nov 7, 2023
@xavieralt xavieralt removed their assignment Nov 8, 2023
@dhrs-odoo dhrs-odoo force-pushed the master-industry-learning-management-services-shag branch 4 times, most recently from e3e7875 to 473854d Compare December 13, 2023 05:43
@@ -0,0 +1,70 @@
# -*- coding: utf-8 -*-
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# -*- coding: utf-8 -*-

{
'name': 'Learning Management Services',
'version': '1.0',
'category': 'Industry',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
'category': 'Industry',
'category': 'Services',

'depends': [
'account_followup',
'appointment_crm',
'base_vat',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
'base_vat',

Need to check

'project_enterprise',
'project_sms',
'sale_crm',
'sale_planning',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
'sale_planning',

'sale_crm',
'sale_planning',
'sale_project',
'web_studio',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
'web_studio',

Comment on lines 16 to 17
<field name="name">Chirag</field>
<field name="email">chka@odoo.com</field>
Copy link
Contributor

Choose a reason for hiding this comment

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

change name and email

Comment on lines 89 to 90
<field name="name">payment</field>
<field name="email">da@da.com</field>
Copy link
Contributor

Choose a reason for hiding this comment

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

Give proper name and factive email / phone

Comment on lines 30 to 42
<record id="sale_advance_payment_inv_1" model="sale.advance.payment.inv">
<field name="advance_payment_method" >delivered</field>
<field name="sale_order_ids" eval="[Command.set([ref('sale_order_2')])]"/>
</record>
<function model="sale.advance.payment.inv" name="create_invoices"
eval="[ref('sale_advance_payment_inv_1')]"/>

<record id="sale_advance_payment_inv_2" model="sale.advance.payment.inv">
<field name="advance_payment_method" >delivered</field>
<field name="sale_order_ids" eval="[Command.set([ref('sale_order_3')])]"/>
</record>
<function model="sale.advance.payment.inv" name="create_invoices"
eval="[ref('sale_advance_payment_inv_2')]"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<record id="sale_advance_payment_inv_1" model="sale.advance.payment.inv">
<field name="advance_payment_method" >delivered</field>
<field name="sale_order_ids" eval="[Command.set([ref('sale_order_2')])]"/>
</record>
<function model="sale.advance.payment.inv" name="create_invoices"
eval="[ref('sale_advance_payment_inv_1')]"/>
<record id="sale_advance_payment_inv_2" model="sale.advance.payment.inv">
<field name="advance_payment_method" >delivered</field>
<field name="sale_order_ids" eval="[Command.set([ref('sale_order_3')])]"/>
</record>
<function model="sale.advance.payment.inv" name="create_invoices"
eval="[ref('sale_advance_payment_inv_2')]"/>
<function name="_create_invoices" model="sale.order">
<value eval="[ref('sale_order_2'), ref('sale_order_3')]"/>
</function>

What if I directly use this function instead of this

<record id="website_menu_1" model="website.menu">
<field name="name">Appointment</field>
<field name="website_id" ref="website.default_website"/>
<field name="url" eval="'appointment/' + str(ref('learning_management_services.appointment_type_1'))"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<field name="url" eval="'appointment/' + str(ref('learning_management_services.appointment_type_1'))"/>
<field name="url" eval="'appointment/' + str(ref('appointment_type_1'))"/>

Comment on lines 22 to 23
Make your finances reach a new level with our online management system. <br />Track every penny, directly from the confort of your home.
</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

correct indentation

@dhrs-odoo dhrs-odoo force-pushed the master-industry-learning-management-services-shag branch from 83eaea1 to 57be14f Compare December 13, 2023 12:19
'appointment_crm',
'knowledge',
'project_enterprise',
'project_sms',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
'project_sms',

Auto installed with project_enterprise?

learning_management_services/__manifest__.py Show resolved Hide resolved
<record id="appointment_type_1" model="appointment.type">
<field name="name">Free e-Learn Skill Share</field>
<field name="is_published" eval="True"/>
<field name="avatars_display">hide</field>
Copy link
Contributor

Choose a reason for hiding this comment

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

pending

<field name="is_published" eval="True"/>
<field name="avatars_display">hide</field>
<field name="lead_create" eval="True"/>
<field name="staff_user_ids" eval="[(6, 0, [ref('base.user_admin')])]"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

pending

<?xml version='1.0' encoding='UTF-8'?>
<odoo>
<record id="knowledge_article_1" model="knowledge.article">
<field name="name"> E-LEARNING MANAGEMENT SERVICE </field>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<field name="name"> E-LEARNING MANAGEMENT SERVICE </field>
<field name="name">E-LEARNING MANAGEMENT SERVICE</field>

<field name="add_to_cart_action">go_to_cart</field>
<field name="favicon" type="base64" file="learning_management_services/static/src/binary/website/1-favicon"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<field name="add_to_cart_action">go_to_cart</field>
<field name="favicon" type="base64" file="learning_management_services/static/src/binary/website/1-favicon"/>
<field name="favicon" type="base64" file="learning_management_services/static/src/binary/website/1-favicon"/>
<field name="add_to_cart_action">go_to_cart</field>

@@ -0,0 +1,464 @@
<?xml version='1.0' encoding='UTF-8'?>
<odoo>
<record id="learning_management_services.homepage" model="ir.ui.view">
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<record id="learning_management_services.homepage" model="ir.ui.view">
<record id="homepage" model="ir.ui.view">

</field>
</record>

<record id="learning_management_services.contactus" model="ir.ui.view">
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<record id="learning_management_services.contactus" model="ir.ui.view">
<record id="contactus" model="ir.ui.view">

<field name="key">website_crm.contactus_form</field>
<field name="type">qweb</field>
<field name="mode">extension</field>
<field name="inherit_id" ref="learning_management_services.contactus" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<field name="inherit_id" ref="learning_management_services.contactus" />
<field name="inherit_id" ref="contactus" />

<field name="public" eval="True"/>
<field name="website_id" ref="website.default_website"/>
</record>
<record id="learning_management_services.configurator_1_s_banner_default_image" model="ir.attachment">
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<record id="learning_management_services.configurator_1_s_banner_default_image" model="ir.attachment">
<record id="configurator_1_s_banner_default_image" model="ir.attachment">

@shag-odoo shag-odoo force-pushed the master-industry-learning-management-services-shag branch from 57be14f to 6c92241 Compare December 18, 2023 13:10
@shag-odoo
Copy link
Contributor

Hello @paso-odoo
All requested changes have been implemented

@shag-odoo shag-odoo force-pushed the master-industry-learning-management-services-shag branch from 6c92241 to 76bdad5 Compare December 18, 2023 13:20
Comment on lines 6 to 8
<!-- <field name="avatars_display">hide</field> -->
<field name="lead_create" eval="True"/>
<!-- <field name="staff_user_ids" eval="[(6, 0, [ref('base.user_admin')])]"/> -->
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<!-- <field name="avatars_display">hide</field> -->
<field name="lead_create" eval="True"/>
<!-- <field name="staff_user_ids" eval="[(6, 0, [ref('base.user_admin')])]"/> -->
<field name="lead_create" eval="True"/>

<li id="checkId-5">Methodology</li>
<li id="checkId-6">Payment Process</li>
</ul>
<p><br /></p>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<p><br /></p>

Comment on lines 14 to 21
<p strong><font style="font-size: 27pt">E-LEARNING
MANAGEMENT SERVICE</font></strong>
</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<p strong><font style="font-size: 27pt">E-LEARNING
MANAGEMENT SERVICE</font></strong>
</p>
<p strong><font style="font-size: 27pt">E-LEARNING MANAGEMENT SERVICE</font></strong></p>

<p strong><font style="font-size: 27pt">E-LEARNING
MANAGEMENT SERVICE</font></strong>
</p>
<p style="margin-bottom: 0px;">
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<p style="margin-bottom: 0px;">
<p>

apply in whole article

<p style="margin-bottom: 0px;">
The online e-learning management service industry is a rapidly growing sector that involves the delivery of educational content and training programs through digital platforms.&nbsp;
</p>
<p style="margin-bottom: 0px;">Also they provide the infrastructure for delivering, managing, and tracking online courses. They offer features such as course creation, student enrolment, assessments, progress tracking, communication tools, certification and gamification.</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<p style="margin-bottom: 0px;">Also they provide the infrastructure for delivering, managing, and tracking online courses. They offer features such as course creation, student enrolment, assessments, progress tracking, communication tools, certification and gamification.</p>
<p style="margin-bottom: 0px;">Also, they provide the infrastructure for delivering, managing, and tracking online courses. They offer features such as course creation, student enrolment, assessments, progress tracking, communication tools, certification and gamification.</p>

<?xml version='1.0' encoding='UTF-8'?>
<odoo>
<!-- confirm sale order-->
<function model="sale.order" name="action_confirm">
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<function model="sale.order" name="action_confirm">
<function name="action_confirm" model="sale.order">

<?xml version='1.0' encoding='UTF-8'?>
<odoo noupdate="1">
<record id="sale_order_line_1" model="sale.order.line">
<field name="name">Access to: Become An expert in Digital Marketing..</field>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<field name="name">Access to: Become An expert in Digital Marketing..</field>

</h2>
</div>
<div class="col-lg-6 pt16 pb16">
<img src="/web_editor/image_shape/learning_management_services.configurator_1_s_text_image_default_image/web_editor/pattern/pattern_point.svg?c1=o-color-1" class="img img-fluid mx-auto" alt="" data-shape="web_editor/pattern/pattern_point" data-original-mimetype="image/jpeg" data-file-name="image_content_19.svg" data-shape-colors="o-color-1;;;;" loading="lazy" style="" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<img src="/web_editor/image_shape/learning_management_services.configurator_1_s_text_image_default_image/web_editor/pattern/pattern_point.svg?c1=o-color-1" class="img img-fluid mx-auto" alt="" data-shape="web_editor/pattern/pattern_point" data-original-mimetype="image/jpeg" data-file-name="image_content_19.svg" data-shape-colors="o-color-1;;;;" loading="lazy" style="" />
<img src="/web_editor/image_shape/learning_management_services.configurator_1_s_text_image_default_image/web_editor/pattern/pattern_point.svg?c1=o-color-1" class="img img-fluid mx-auto" data-shape="web_editor/pattern/pattern_point" data-original-mimetype="image/jpeg" data-file-name="image_content_19.svg" data-shape-colors="o-color-1;;;;" loading="lazy"/>

<div class="carousel-item pt256 pb256 o_cc o_cc3 o_colored_level oe_img_bg o_bg_img_center active" style="background-image: url('/web/image/learning_management_services.ir_attachment_971'); position: relative; background-position: 50% 65%;" data-name="Slide" data-original-id="961" data-original-src="/unsplash/FlPc9_VocJ4/elearn.jpg?unique=4cbc548a" data-mimetype="image/webp" data-resize-width="1920">
<div class="o_we_bg_filter bg-black-50" />
<div class="container">

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

<field name="arch" type="xml">
<xpath expr="//*[@id='o_fake_navbar_brand']" position="replace">
<a href="/" t-attf-class="navbar-brand logo #{_link_class}">

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

@shag-odoo shag-odoo force-pushed the master-industry-learning-management-services-shag branch from 76bdad5 to 9eb1e55 Compare December 27, 2023 05:48
@shag-odoo shag-odoo changed the base branch from 17.0 to master January 8, 2024 06:29
@shag-odoo shag-odoo changed the base branch from master to 17.0 January 8, 2024 06:33
@shag-odoo shag-odoo force-pushed the master-industry-learning-management-services-shag branch 2 times, most recently from b02a0eb to c546ae0 Compare January 8, 2024 06:48
@shag-odoo shag-odoo force-pushed the master-industry-learning-management-services-shag branch 2 times, most recently from 350a51f to ac26885 Compare January 22, 2024 05:19
Copy link
Contributor

@prra-odoo prra-odoo left a comment

Choose a reason for hiding this comment

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

Hello @shag-odoo

I have posted my initial review!
Can you please have a look at it?

Thanks!

learning_management_services/data/knowledge_article.xml Outdated Show resolved Hide resolved
learning_management_services/data/knowledge_article.xml Outdated Show resolved Hide resolved
learning_management_services/data/knowledge_article.xml Outdated Show resolved Hide resolved
learning_management_services/data/knowledge_article.xml Outdated Show resolved Hide resolved
learning_management_services/data/product_product.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website_view.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website_view.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website_view.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website_view.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website_view.xml Outdated Show resolved Hide resolved
@shag-odoo
Copy link
Contributor

Hello @prra-odoo
Thanks for the suggestions, Changes are done and few suggestions to your feedback

Have a great day ahead 😇

Copy link
Contributor

@prra-odoo prra-odoo left a comment

Choose a reason for hiding this comment

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

Hello @shag-odoo

Thanks for working on this!
Some final remarks from my side.
Can you please have a look at it?

learning_management_services/__manifest__.py Outdated Show resolved Hide resolved
learning_management_services/data/knowledge_article.xml Outdated Show resolved Hide resolved
learning_management_services/data/knowledge_article.xml Outdated Show resolved Hide resolved
learning_management_services/data/knowledge_article.xml Outdated Show resolved Hide resolved
learning_management_services/data/product_category.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website_view.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website_view.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website_view.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website_view.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website_view.xml Outdated Show resolved Hide resolved
@shag-odoo shag-odoo force-pushed the master-industry-learning-management-services-shag branch from 57f009b to 56bb290 Compare January 25, 2024 09:12
@arpa-odoo arpa-odoo force-pushed the master-industry-learning-management-services-shag branch 4 times, most recently from cc2c334 to 9d397aa Compare April 26, 2024 09:23
Copy link
Collaborator

@vava-odoo vava-odoo left a comment

Choose a reason for hiding this comment

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

Thanks @arpa-odoo for the work on this. Please find below a few comments.
Cheers!

learning_management_services/data/product_pricelist.xml Outdated Show resolved Hide resolved
learning_management_services/data/product_product.xml Outdated Show resolved Hide resolved
learning_management_services/data/slide_channel.xml Outdated Show resolved Hide resolved
learning_management_services/data/slide_channel.xml Outdated Show resolved Hide resolved
learning_management_services/demo/sale_order_confirm.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website_view.xml Outdated Show resolved Hide resolved
learning_management_services/demo/website_view.xml Outdated Show resolved Hide resolved
@arpa-odoo arpa-odoo force-pushed the master-industry-learning-management-services-shag branch from 9d397aa to d41ff22 Compare May 13, 2024 05:55
@arpa-odoo arpa-odoo force-pushed the master-industry-learning-management-services-shag branch from d41ff22 to aa977ed Compare May 15, 2024 08:59
Comment on lines +326 to +329
}
"/>
<t t-set="contactus_form_values" t-value="dict(contactus_form_values,
**{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
}
"/>
<t t-set="contactus_form_values" t-value="dict(contactus_form_values,
**{

Comment on lines +56 to +76
obj().env.ref('learning_management_services.sale_order_2') +
obj().env.ref('learning_management_services.sale_order_3') +
obj().env.ref('learning_management_services.sale_order_4') +
obj().env.ref('learning_management_services.sale_order_5') +
obj().env.ref('learning_management_services.sale_order_6') +
obj().env.ref('learning_management_services.sale_order_7') +
obj().env.ref('learning_management_services.sale_order_8') +
obj().env.ref('learning_management_services.sale_order_9') +
obj().env.ref('learning_management_services.sale_order_10') +
obj().env.ref('learning_management_services.sale_order_11') +
obj().env.ref('learning_management_services.sale_order_12') +
obj().env.ref('learning_management_services.sale_order_14') +
obj().env.ref('learning_management_services.sale_order_16') +
obj().env.ref('learning_management_services.sale_order_17') +
obj().env.ref('learning_management_services.sale_order_18') +
obj().env.ref('learning_management_services.sale_order_21') +
obj().env.ref('learning_management_services.sale_order_22') +
obj().env.ref('learning_management_services.sale_order_34') +
obj().env.ref('learning_management_services.sale_order_35') +
obj().env.ref('learning_management_services.sale_order_36')
).invoice_ids.ids
Copy link
Collaborator

Choose a reason for hiding this comment

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

See 0016d23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready PR is ready for review and integration
Projects
None yet
9 participants