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

feat: #9 change the plugin name #21

Open
wants to merge 58 commits into
base: main
Choose a base branch
from

Conversation

SalahAdDin
Copy link

What does it do?

It renames the plugin from the annoying and long strapi-plugin-rich-text to richtext-editor in the package strapi property. It does not affect the repository name, folder name, or plugin Id.

Why is it needed?

strapi-plugin-rich-text looks pretty ugly in the plugin configuration file and URLs. We chose richtext-editor so it is pretty clear what the plugin does in the plugin file or the URLs.

How to test it?

Just enable the plugin:
plugin.ts:

export default {
  "richtext-editor": {
    enabled: true,
    resolve: "./node_modules/strapi-plugin-rich-text",
  },
};

And check the available routes:

pnpm strapi routes:list                    

> strapi@0.1.0 strapi /media/Storage/Projects/Experiments/opensource/strapi-plugin-rich-text/apps/strapi
> strapi "routes:list"

├────────────────────┼─────────────────────────────────────────────────────────────────────────────────┤
 HEAD|GET            /richtext-editor                                                                
├────────────────────┼─────────────────────────────────────────────────────────────────────────────────┤

Notes

fix: adding scroll to editor content
chore: adding localization to the editor
feat: character and words count display
feat: adding Alert extension to the editor
fix: adding missing ", on es localization file
feat: #7 add color, highlight extensions and fix text align buttons
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant