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 custom composer for polling feature #38

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

Conversation

bobofishbo
Copy link
Contributor

Add the nodebb-plugin-composer-custom directory, which is a replication of the original built-in nodebb-plugin-composer-default, but with changes for polling and previewing polls while maintaining all previous functionalities of the default composer.

This includes writing nodebb-plugin-composer-custom/static/templates/partials/poll-modal.tpl, which generates a modal asking user for poll title, poll options. Changes in the nodebb-plugin-composer-custom/static/lib/client.js file allow for the adding and deleting of options, as well as saving the poll into the default modal. Preview and edit all work as it would in the default composer.

To test changes, one must un-link from the default composer and switch to the custom composer. Changes would include

  1. src/meta/build.js: switch 2 instances of composer-default to composer-custom
  2. src/install.js: switch 1 instance of composer-default to composer-custom
  3. switch package.json line with composer-default to "nodebb-plugin-composer-custom": "file:nodebb-plugin-composer-custom",

For easier demonstration of the functionality, see this demo video:
https://drive.google.com/file/d/1VxbekzTjQcDLr4_fYX84Mjl-UPuUhHAf/view?usp=sharing

@bobofishbo bobofishbo added the enhancement New feature or request label Feb 27, 2025
@bobofishbo
Copy link
Contributor Author

demo_bo.mov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant