jaljo / rich-text-editor Goto Github PK

The current project architecture is a bit confusing, because the server components are well isolated in a dedicated folder, whereas the front components are directly at the root of the repository:

rich-text-editor
|_ server
|_ public
|_ src

We should introduce a new level of hierarchy for the front project like in the following example:

rich-text-editor
|_ server
|_ app
    |_ public
    |_ src

The video insertion formerly depended on brightcove. Maybe we can consider using another API (pexels provides a video API too), to bring this feature back from the dead.

Because the codebase has been extracted from a wider project years ago, there is a lot of dead code and unused imports. Enabling eslint on the project would be of great help to improve the codebase.

The tweet insertion has been disabled because the server layer, used to render tweet, can no longer be built. Some changes have to be done on the docker configuration in order to correctly build the express layer, and re-enable this feature.

Now eslint has been enabled on the project, those error should be fixed. Consider creating separated PR's for each rule, as this is likely to lead to massive refactoring.

Pull requests:
#15