effective_email_templates 0.5.1 → 1.0.0
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/MIT-LICENSE +1 -1
- data/README.md +22 -36
- data/app/controllers/admin/email_templates_controller.rb +25 -16
- data/app/datatables/effective_email_templates_datatable.rb +18 -16
- data/app/mailers/effective/email_templates_mailer.rb +22 -0
- data/app/models/effective/email_template.rb +66 -34
- data/app/views/admin/email_templates/_actions.html.haml +5 -2
- data/app/views/admin/email_templates/_form.html.haml +16 -9
- data/app/views/admin/email_templates/index.html.haml +1 -1
- data/app/views/layouts/effective_email_templates_mailer_layout.html.haml +7 -0
- data/config/effective_email_templates.rb +16 -25
- data/config/routes.rb +6 -8
- data/db/migrate/01_create_effective_email_templates.rb.erb +14 -8
- data/lib/effective_email_templates.rb +18 -14
- data/lib/effective_email_templates/engine.rb +1 -1
- data/lib/effective_email_templates/importer.rb +55 -0
- data/lib/effective_email_templates/version.rb +1 -1
- data/lib/generators/effective_email_templates/install_generator.rb +1 -1
- data/lib/tasks/effective_email_templates_tasks.rake +19 -0
- metadata +30 -142
- data/Rakefile +0 -23
- data/app/assets/javascripts/effective_email_templates.js +0 -1
- data/app/assets/stylesheets/effective_email_templates.css.scss +0 -1
- data/app/helpers/effective_email_templates_helper.rb +0 -19
- data/app/mailers/effective/email_template_mailer.rb +0 -14
- data/app/models/effective/datatables/email_templates.rb +0 -24
- data/app/views/effective/email_template_mailer/templated_email.html.haml +0 -1
- data/lib/effective/liquid_mailer.rb +0 -15
- data/lib/effective_email_templates/email_view_template.rb +0 -39
- data/lib/effective_email_templates/liquid_resolver.rb +0 -46
- data/lib/effective_email_templates/template_importer.rb +0 -51
- data/lib/tasks/effective_email_templates/import_default_views.rake +0 -19
- data/spec/controllers/admin/email_templates_controller_spec.rb +0 -60
- data/spec/dummy/README.rdoc +0 -28
- data/spec/dummy/Rakefile +0 -6
- data/spec/dummy/app/assets/javascripts/application.js +0 -13
- data/spec/dummy/app/assets/stylesheets/application.css +0 -15
- data/spec/dummy/app/controllers/application_controller.rb +0 -5
- data/spec/dummy/app/controllers/welcome_controller.rb +0 -4
- data/spec/dummy/app/helpers/application_helper.rb +0 -2
- data/spec/dummy/app/mailers/liquid_resolved_mailer.rb +0 -9
- data/spec/dummy/app/mailers/user_liquid_mailer.rb +0 -10
- data/spec/dummy/app/models/user.rb +0 -15
- data/spec/dummy/app/views/layouts/application.html.erb +0 -17
- data/spec/dummy/app/views/user_liquid/after_create_user.liquid +0 -7
- data/spec/dummy/app/views/welcome/index.html.haml +0 -1
- data/spec/dummy/bin/bundle +0 -3
- data/spec/dummy/bin/rails +0 -4
- data/spec/dummy/bin/rake +0 -4
- data/spec/dummy/config.ru +0 -4
- data/spec/dummy/config/application.rb +0 -26
- data/spec/dummy/config/boot.rb +0 -5
- data/spec/dummy/config/database.yml +0 -25
- data/spec/dummy/config/environment.rb +0 -5
- data/spec/dummy/config/environments/development.rb +0 -37
- data/spec/dummy/config/environments/production.rb +0 -78
- data/spec/dummy/config/environments/test.rb +0 -40
- data/spec/dummy/config/initializers/assets.rb +0 -8
- data/spec/dummy/config/initializers/backtrace_silencers.rb +0 -7
- data/spec/dummy/config/initializers/cookies_serializer.rb +0 -3
- data/spec/dummy/config/initializers/devise.rb +0 -259
- data/spec/dummy/config/initializers/effective_email_templates.rb +0 -51
- data/spec/dummy/config/initializers/filter_parameter_logging.rb +0 -4
- data/spec/dummy/config/initializers/inflections.rb +0 -16
- data/spec/dummy/config/initializers/mime_types.rb +0 -4
- data/spec/dummy/config/initializers/session_store.rb +0 -3
- data/spec/dummy/config/initializers/wrap_parameters.rb +0 -14
- data/spec/dummy/config/locales/devise.en.yml +0 -60
- data/spec/dummy/config/locales/en.yml +0 -23
- data/spec/dummy/config/routes.rb +0 -57
- data/spec/dummy/config/secrets.yml +0 -22
- data/spec/dummy/db/migrate/20141126222940_devise_create_users.rb +0 -42
- data/spec/dummy/db/migrate/20141126222941_create_effective_email_templates.rb +0 -20
- data/spec/dummy/db/schema.rb +0 -46
- data/spec/dummy/public/404.html +0 -67
- data/spec/dummy/public/422.html +0 -67
- data/spec/dummy/public/500.html +0 -66
- data/spec/dummy/public/favicon.ico +0 -0
- data/spec/effective_email_templates_spec.rb +0 -35
- data/spec/factories/email_template.rb +0 -12
- data/spec/factories/user.rb +0 -16
- data/spec/factory_spec.rb +0 -10
- data/spec/lib/effective_email_templates/template_importer_spec.rb +0 -73
- data/spec/mailers/liquid_resolved_mailer_spec.rb +0 -38
- data/spec/models/email_template_spec.rb +0 -61
- data/spec/models/user_spec.rb +0 -10
- data/spec/sanity_spec.rb +0 -7
- data/spec/spec_helper.rb +0 -26
data/spec/dummy/public/422.html
DELETED
@@ -1,67 +0,0 @@
|
|
1
|
-
<!DOCTYPE html>
|
2
|
-
<html>
|
3
|
-
<head>
|
4
|
-
<title>The change you wanted was rejected (422)</title>
|
5
|
-
<meta name="viewport" content="width=device-width,initial-scale=1">
|
6
|
-
<style>
|
7
|
-
body {
|
8
|
-
background-color: #EFEFEF;
|
9
|
-
color: #2E2F30;
|
10
|
-
text-align: center;
|
11
|
-
font-family: arial, sans-serif;
|
12
|
-
margin: 0;
|
13
|
-
}
|
14
|
-
|
15
|
-
div.dialog {
|
16
|
-
width: 95%;
|
17
|
-
max-width: 33em;
|
18
|
-
margin: 4em auto 0;
|
19
|
-
}
|
20
|
-
|
21
|
-
div.dialog > div {
|
22
|
-
border: 1px solid #CCC;
|
23
|
-
border-right-color: #999;
|
24
|
-
border-left-color: #999;
|
25
|
-
border-bottom-color: #BBB;
|
26
|
-
border-top: #B00100 solid 4px;
|
27
|
-
border-top-left-radius: 9px;
|
28
|
-
border-top-right-radius: 9px;
|
29
|
-
background-color: white;
|
30
|
-
padding: 7px 12% 0;
|
31
|
-
box-shadow: 0 3px 8px rgba(50, 50, 50, 0.17);
|
32
|
-
}
|
33
|
-
|
34
|
-
h1 {
|
35
|
-
font-size: 100%;
|
36
|
-
color: #730E15;
|
37
|
-
line-height: 1.5em;
|
38
|
-
}
|
39
|
-
|
40
|
-
div.dialog > p {
|
41
|
-
margin: 0 0 1em;
|
42
|
-
padding: 1em;
|
43
|
-
background-color: #F7F7F7;
|
44
|
-
border: 1px solid #CCC;
|
45
|
-
border-right-color: #999;
|
46
|
-
border-left-color: #999;
|
47
|
-
border-bottom-color: #999;
|
48
|
-
border-bottom-left-radius: 4px;
|
49
|
-
border-bottom-right-radius: 4px;
|
50
|
-
border-top-color: #DADADA;
|
51
|
-
color: #666;
|
52
|
-
box-shadow: 0 3px 8px rgba(50, 50, 50, 0.17);
|
53
|
-
}
|
54
|
-
</style>
|
55
|
-
</head>
|
56
|
-
|
57
|
-
<body>
|
58
|
-
<!-- This file lives in public/422.html -->
|
59
|
-
<div class="dialog">
|
60
|
-
<div>
|
61
|
-
<h1>The change you wanted was rejected.</h1>
|
62
|
-
<p>Maybe you tried to change something you didn't have access to.</p>
|
63
|
-
</div>
|
64
|
-
<p>If you are the application owner check the logs for more information.</p>
|
65
|
-
</div>
|
66
|
-
</body>
|
67
|
-
</html>
|
data/spec/dummy/public/500.html
DELETED
@@ -1,66 +0,0 @@
|
|
1
|
-
<!DOCTYPE html>
|
2
|
-
<html>
|
3
|
-
<head>
|
4
|
-
<title>We're sorry, but something went wrong (500)</title>
|
5
|
-
<meta name="viewport" content="width=device-width,initial-scale=1">
|
6
|
-
<style>
|
7
|
-
body {
|
8
|
-
background-color: #EFEFEF;
|
9
|
-
color: #2E2F30;
|
10
|
-
text-align: center;
|
11
|
-
font-family: arial, sans-serif;
|
12
|
-
margin: 0;
|
13
|
-
}
|
14
|
-
|
15
|
-
div.dialog {
|
16
|
-
width: 95%;
|
17
|
-
max-width: 33em;
|
18
|
-
margin: 4em auto 0;
|
19
|
-
}
|
20
|
-
|
21
|
-
div.dialog > div {
|
22
|
-
border: 1px solid #CCC;
|
23
|
-
border-right-color: #999;
|
24
|
-
border-left-color: #999;
|
25
|
-
border-bottom-color: #BBB;
|
26
|
-
border-top: #B00100 solid 4px;
|
27
|
-
border-top-left-radius: 9px;
|
28
|
-
border-top-right-radius: 9px;
|
29
|
-
background-color: white;
|
30
|
-
padding: 7px 12% 0;
|
31
|
-
box-shadow: 0 3px 8px rgba(50, 50, 50, 0.17);
|
32
|
-
}
|
33
|
-
|
34
|
-
h1 {
|
35
|
-
font-size: 100%;
|
36
|
-
color: #730E15;
|
37
|
-
line-height: 1.5em;
|
38
|
-
}
|
39
|
-
|
40
|
-
div.dialog > p {
|
41
|
-
margin: 0 0 1em;
|
42
|
-
padding: 1em;
|
43
|
-
background-color: #F7F7F7;
|
44
|
-
border: 1px solid #CCC;
|
45
|
-
border-right-color: #999;
|
46
|
-
border-left-color: #999;
|
47
|
-
border-bottom-color: #999;
|
48
|
-
border-bottom-left-radius: 4px;
|
49
|
-
border-bottom-right-radius: 4px;
|
50
|
-
border-top-color: #DADADA;
|
51
|
-
color: #666;
|
52
|
-
box-shadow: 0 3px 8px rgba(50, 50, 50, 0.17);
|
53
|
-
}
|
54
|
-
</style>
|
55
|
-
</head>
|
56
|
-
|
57
|
-
<body>
|
58
|
-
<!-- This file lives in public/500.html -->
|
59
|
-
<div class="dialog">
|
60
|
-
<div>
|
61
|
-
<h1>We're sorry, but something went wrong.</h1>
|
62
|
-
</div>
|
63
|
-
<p>If you are the application owner check the logs for more information.</p>
|
64
|
-
</div>
|
65
|
-
</body>
|
66
|
-
</html>
|
File without changes
|
@@ -1,35 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
describe EffectiveEmailTemplates do
|
4
|
-
describe '::get' do
|
5
|
-
it 'finds templates by their slug' do
|
6
|
-
template = create(:email_template, slug: 'a_unique_email_template_identifier')
|
7
|
-
expect(
|
8
|
-
EffectiveEmailTemplates.get(:a_unique_email_template_identifier)
|
9
|
-
).to eq( template )
|
10
|
-
end
|
11
|
-
|
12
|
-
context 'when the template does not exist' do
|
13
|
-
before :each do
|
14
|
-
@template = EffectiveEmailTemplates.get(:this_doesnt_exist)
|
15
|
-
end
|
16
|
-
|
17
|
-
it "returns a template object" do
|
18
|
-
expect(@template).to be_a(Effective::EmailTemplate)
|
19
|
-
end
|
20
|
-
|
21
|
-
it 'returns a template object with a slug' do
|
22
|
-
expect(@template.slug).to eq(:this_doesnt_exist)
|
23
|
-
end
|
24
|
-
|
25
|
-
it 'returns a template object without other attributes' do
|
26
|
-
# The template automatically gets generated by Rail's ::serialize method but its an empty template
|
27
|
-
expect(@template.template.render('')).to eq ''
|
28
|
-
attributes = @template.attributes
|
29
|
-
attributes.delete("slug")
|
30
|
-
attributes.delete("template")
|
31
|
-
expect(attributes.values.compact).to be_empty
|
32
|
-
end
|
33
|
-
end
|
34
|
-
end
|
35
|
-
end
|
data/spec/factories/user.rb
DELETED
@@ -1,16 +0,0 @@
|
|
1
|
-
FactoryGirl.define do
|
2
|
-
factory :user do
|
3
|
-
password "password"
|
4
|
-
password_confirmation "password"
|
5
|
-
|
6
|
-
sequence(:email) { |n| "john-#{n}@example.com" }
|
7
|
-
|
8
|
-
before(:create) do |user, evaluator|
|
9
|
-
FactoryGirl.create(:email_template, slug: :after_create_user)
|
10
|
-
end
|
11
|
-
end
|
12
|
-
|
13
|
-
factory :admin, parent: :user do
|
14
|
-
end
|
15
|
-
end
|
16
|
-
|
data/spec/factory_spec.rb
DELETED
@@ -1,73 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
require 'effective_email_templates/template_importer'
|
3
|
-
|
4
|
-
describe EffectiveEmailTemplates::TemplateImporter do
|
5
|
-
before { Effective::EmailTemplate.delete_all }
|
6
|
-
|
7
|
-
describe '.invoke' do
|
8
|
-
context 'without overwriting' do
|
9
|
-
context 'when well formatted template files' do
|
10
|
-
it 'should import templates from view files' do
|
11
|
-
expect { EffectiveEmailTemplates::TemplateImporter.invoke }.to change { Effective::EmailTemplate.count }
|
12
|
-
end
|
13
|
-
|
14
|
-
it 'should not import templates if a template already exists in the database' do
|
15
|
-
expect { EffectiveEmailTemplates::TemplateImporter.invoke }.to change { Effective::EmailTemplate.count }
|
16
|
-
expect { EffectiveEmailTemplates::TemplateImporter.invoke }.to_not change { Effective::EmailTemplate.count }
|
17
|
-
end
|
18
|
-
|
19
|
-
it 'should not print errors' do
|
20
|
-
importer = EffectiveEmailTemplates::TemplateImporter.new
|
21
|
-
expect(importer).to_not receive(:print_errors)
|
22
|
-
EffectiveEmailTemplates::TemplateImporter.invoke(importer)
|
23
|
-
end
|
24
|
-
end
|
25
|
-
|
26
|
-
context 'when poorly formatted template files' do
|
27
|
-
let(:filepath) { Rails.root.join('app', 'views', 'user_liquid', 'some_template.liquid') }
|
28
|
-
|
29
|
-
before { File.open(filepath, 'w') { |f| f.write("--\n---\nbody") } }
|
30
|
-
after { File.delete(filepath) }
|
31
|
-
|
32
|
-
it 'should print errors if there is a problem with a template' do
|
33
|
-
importer = EffectiveEmailTemplates::TemplateImporter.new
|
34
|
-
expect(importer).to receive(:print_errors)
|
35
|
-
EffectiveEmailTemplates::TemplateImporter.invoke(importer)
|
36
|
-
end
|
37
|
-
end
|
38
|
-
end
|
39
|
-
|
40
|
-
context 'with overwriting' do
|
41
|
-
let!(:existing_template) { FactoryGirl.create(:email_template, body: 'test', slug: 'after_create_user') }
|
42
|
-
|
43
|
-
context 'when well formatted template files' do
|
44
|
-
it 'should update templates from view files' do
|
45
|
-
expect { EffectiveEmailTemplates::TemplateImporter.invoke(overwrite: true) }.to change { existing_template.reload.body }.from('test').to('Hello {{ user_name }}')
|
46
|
-
end
|
47
|
-
|
48
|
-
it 'should not create duplicate templates if a template already exists in the database' do
|
49
|
-
expect { EffectiveEmailTemplates::TemplateImporter.invoke(overwrite: true) }.not_to change { Effective::EmailTemplate.count }
|
50
|
-
end
|
51
|
-
|
52
|
-
it 'should not print errors' do
|
53
|
-
importer = EffectiveEmailTemplates::TemplateImporter.new
|
54
|
-
expect(importer).to_not receive(:print_errors)
|
55
|
-
EffectiveEmailTemplates::TemplateImporter.invoke(importer, overwrite: true)
|
56
|
-
end
|
57
|
-
end
|
58
|
-
|
59
|
-
context 'when poorly formatted template files' do
|
60
|
-
let(:filepath) { Rails.root.join('app', 'views', 'user_liquid', 'some_template.liquid') }
|
61
|
-
|
62
|
-
before { File.open(filepath, 'w') { |f| f.write("--\n---\nbody") } }
|
63
|
-
after { File.delete(filepath) }
|
64
|
-
|
65
|
-
it 'should print errors if there is a problem with a template' do
|
66
|
-
importer = EffectiveEmailTemplates::TemplateImporter.new
|
67
|
-
expect(importer).to receive(:print_errors)
|
68
|
-
EffectiveEmailTemplates::TemplateImporter.invoke(importer, overwrite: true)
|
69
|
-
end
|
70
|
-
end
|
71
|
-
end
|
72
|
-
end
|
73
|
-
end
|
@@ -1,38 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
describe LiquidResolvedMailer do
|
4
|
-
|
5
|
-
describe 'basic template resolution and rendering' do
|
6
|
-
before :each do
|
7
|
-
@email_body = 'liquid resolution!'
|
8
|
-
@template = create(:email_template, slug: 'test_email', body: @email_body)
|
9
|
-
@mail = LiquidResolvedMailer.test_email
|
10
|
-
end
|
11
|
-
|
12
|
-
it 'creates emails using a liquid template' do
|
13
|
-
expect(@mail.body.to_s).to eq(@email_body)
|
14
|
-
end
|
15
|
-
end
|
16
|
-
|
17
|
-
describe 'embedded template resolution and rendering' do
|
18
|
-
it 'correctly renders emails using an embedded liquid template' do
|
19
|
-
@email_body = 'liquid {{ noun }}!'
|
20
|
-
@renderred_body = 'liquid resolution!'
|
21
|
-
@template = create(:email_template, slug: 'test_email', body: @email_body)
|
22
|
-
@mail = LiquidResolvedMailer.test_email
|
23
|
-
expect(@mail.body.to_s).to eq(@renderred_body)
|
24
|
-
end
|
25
|
-
|
26
|
-
it 'renders emails with nested variables' do
|
27
|
-
@email_body = 'liquid {{ noun.for_the_new_year }}!'
|
28
|
-
@renderred_body = 'liquid resolution!'
|
29
|
-
@template = create(:email_template, slug: 'test_email', body: @email_body)
|
30
|
-
@mail = LiquidResolvedMailer.test_email({
|
31
|
-
'noun' => {
|
32
|
-
'for_the_new_year' => 'resolution'
|
33
|
-
}
|
34
|
-
})
|
35
|
-
expect(@mail.body.to_s).to eq(@renderred_body)
|
36
|
-
end
|
37
|
-
end
|
38
|
-
end
|
@@ -1,61 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
describe Effective::EmailTemplate do
|
4
|
-
before :each do
|
5
|
-
@email = create(:email_template)
|
6
|
-
end
|
7
|
-
|
8
|
-
describe 'slug' do
|
9
|
-
it 'can have numbers in the slug' do
|
10
|
-
email = build(:email_template, slug: 'reminder_30_days')
|
11
|
-
expect(email).to be_valid
|
12
|
-
end
|
13
|
-
|
14
|
-
it 'cannot have spaces in the slug' do
|
15
|
-
email = build(:email_template, slug: 'reminder 30 days')
|
16
|
-
expect(email).to_not be_valid
|
17
|
-
end
|
18
|
-
end
|
19
|
-
|
20
|
-
it 'should be persisted' do
|
21
|
-
expect(@email.persisted?).to be(true)
|
22
|
-
end
|
23
|
-
|
24
|
-
it 'can be valid without a template explicitly stored' do
|
25
|
-
email = build(:email_template, template: nil)
|
26
|
-
expect(email).to be_valid
|
27
|
-
end
|
28
|
-
|
29
|
-
it 'stores a template after precompiling' do
|
30
|
-
email = build(:email_template)
|
31
|
-
|
32
|
-
# The initial template is an empty liquid template provided by the ::serialize method
|
33
|
-
expect(email.template.render).to be_blank
|
34
|
-
|
35
|
-
email.precompile
|
36
|
-
|
37
|
-
expect(email.template.render).not_to be_blank
|
38
|
-
end
|
39
|
-
|
40
|
-
it 'stores a precompiled template' do
|
41
|
-
expect(@email.template).to be_a(Liquid::Template)
|
42
|
-
end
|
43
|
-
|
44
|
-
it 'knows how to render itself' do
|
45
|
-
expect(@email.render).to be_a(String)
|
46
|
-
end
|
47
|
-
|
48
|
-
it 'does not precompile if the body has not changed' do
|
49
|
-
@email.from = 'other@example.com'
|
50
|
-
expect(Liquid::Template).to_not receive(:parse).with(@email.body)
|
51
|
-
@email.save
|
52
|
-
end
|
53
|
-
|
54
|
-
it 'does precompile if the body has changed' do
|
55
|
-
@email.body = 'Hello World -- changed'
|
56
|
-
parsed_template = Liquid::Template.parse(@email.body)
|
57
|
-
expect(Liquid::Template).to receive(:parse).with(@email.body).and_return(parsed_template)
|
58
|
-
@email.save
|
59
|
-
end
|
60
|
-
end
|
61
|
-
|
data/spec/models/user_spec.rb
DELETED
data/spec/sanity_spec.rb
DELETED
data/spec/spec_helper.rb
DELETED
@@ -1,26 +0,0 @@
|
|
1
|
-
ENV['RAILS_ENV'] ||= 'test'
|
2
|
-
|
3
|
-
require File.expand_path("../dummy/config/environment.rb", __FILE__)
|
4
|
-
# Not sure if I need this
|
5
|
-
# ActiveRecord::Migrator.migrations_paths = [File.expand_path("../dummy/db/migrate", __FILE__)]
|
6
|
-
|
7
|
-
require 'rspec/rails'
|
8
|
-
require 'rspec/autorun'
|
9
|
-
require 'factory_girl_rails'
|
10
|
-
require 'pry'
|
11
|
-
|
12
|
-
Rails.backtrace_cleaner.remove_silencers!
|
13
|
-
|
14
|
-
# Load support files
|
15
|
-
Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
|
16
|
-
|
17
|
-
RSpec.configure do |config|
|
18
|
-
config.mock_with :rspec
|
19
|
-
config.use_transactional_fixtures = true
|
20
|
-
config.infer_base_class_for_anonymous_controllers = false
|
21
|
-
config.infer_spec_type_from_file_location!
|
22
|
-
config.order = "random"
|
23
|
-
config.include Devise::TestHelpers, type: :controller
|
24
|
-
config.include FactoryGirl::Syntax::Methods
|
25
|
-
end
|
26
|
-
|