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

Refactored the Newsletter and User Controller #30

Open
wants to merge 1 commit into
base: engine
Choose a base branch
from

Commits on Jan 13, 2021

  1. Refactored the Newsletter Mails and User Controller

    - Updated Newsletter index action method to show newsletter by type
    - Created single action method to send monthly, opt-in and opt-out
    newsletter mails
    - Updated routes according to the below methods
    - Removed the global send mail and send test buttons and added buttons below individual newsletters.
    - Added before filters and flash messages in Newsletter and User controller
    - Added alert message before deleting newsletter
    - Modified the UI for test mail template
    - Changed the query for fetching newsletter in opt-in worker, opt-out worker and user worker
    - Fix test configuration
    - Fix the bootstrap navbar menu-items active-state issue
    - Fix the gemfile for test environment
    - Removed duplicated methods
    - Updated the depricated deliver method
    - Fix and modified the test cases for User and Newsletter controller
    - Rename the newsletter type constant
    - Fix the user form
    tejaszambre authored and Ashish944 committed Jan 13, 2021
    Configuration menu
    Copy the full SHA
    3475996 View commit details
    Browse the repository at this point in the history