Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove old React version and import React, ReactDOM and createReactCl… #3

Merged
merged 1 commit into from
Nov 16, 2019

Conversation

yelizariev
Copy link

Copy link
Collaborator

@itpp-bot itpp-bot left a comment

Choose a reason for hiding this comment

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

├─ package-lock.json
├─ package.json
└─ ui/
   ├─ assets/
   |  ├─ excluded_files.tpl.html
   |  ├─ index.tpl.html
   |  └─ js/
   |     ├─ excluded_files.js
   |     ├─ hound.js
   |     └─ react-0.12.2.min.js
   ├─ bindata.go
   ├─ content.go
   └─ ui.go

Installable modules remain unchanged.

Not installable modules remain unchanged.

No new features in doc/changelog.rst files of installable modules

sent by ✌️ Odoo Review Bot

…ass directly from the entry-points

In this commit (af4d9a8) React and ReactDOM where added as dependencies of the project
But it seems that they are not used because no entry-point import them. The React version that is used is the version 0.12.2 that is coming from 2014 and it is loaded as a separate script directly from the HTML templates.
The purpose of this commit is to import React, ReactDOM, and createReactClass from the entry-points and get rid of the old react version.
In this way, the project can be escalated using the new features of React
@yelizariev yelizariev merged commit 8525cd9 into itpp-labs:master Nov 16, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants