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.

Sign up for GitHub

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

Jump to bottom

Add a button to send a newsletter to the admin #59

Open

ElviaBth wants to merge 14 commits into develop from feature/send_email_test_for_newsletters

Collaborator

ElviaBth commented May 25, 2023

No description provided.


          add test send a test mail to the logged user

5840d1f

microstudi assigned ElviaBth

microstudi reviewed

View reviewed changes

decidim-admin/app/controllers/decidim/admin/newsletters_controller.rb Outdated Show resolved Hide resolved

microstudi reviewed

View reviewed changes

decidim-admin/spec/system/admin_manages_newsletters_spec.rb Outdated Show resolved Hide resolved

microstudi reviewed

View reviewed changes

decidim-admin/spec/system/admin_manages_newsletters_spec.rb Show resolved Hide resolved

microstudi reviewed

View reviewed changes

decidim-admin/app/views/decidim/admin/newsletters/show.html.erb Show resolved Hide resolved

microstudi reviewed

View reviewed changes

decidim-admin/app/views/decidim/admin/newsletters/index.html.erb Outdated Show resolved Hide resolved

ElviaBth added 4 commits

May 25, 2023 17:52


          test it will check email has been sent

c41951a


          test it will check email has been sent

84953f2


          test admin send emails from index

25f3849


          test admin receives the test newsletter

1fb9953

ElviaBth changed the title ~~add test send a test mail to the logged user~~ Add a button to send a newsletter to the admin

microstudi and others added 9 commits

June 14, 2023 12:30


          Merge remote-tracking branch 'origin/develop' into feature/send_email…

bdd28bc

…_test_for_newsletters


          Updated code sintax

2d3cc74


          Merge branch 'feature/send_email_test_for_newsletters' of github.com:…

29ae681

…openpoke/decidim into feature/send_email_test_for_newsletters


          Update code after review suggestions

33bc143


          remove restriction to send a newsletter email

fb9eba4


          fix string error in newsletters test

e23b95f


          add context to test title and remove yarn.lock file

2f3f483


          add context to test description

f688005


          replace if ! condition by unless

c414c26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet