Skip to content

Commit

Permalink
review(JOB-170): review by PTR
Browse files Browse the repository at this point in the history
  • Loading branch information
@VictorEnaud
VictorEnaud committed Nov 29, 2017
1 parent 0eb2227 commit 55948d8
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 13 deletions.
2 changes: 1 addition & 1 deletion server/package.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@
"sequelize": "^4.4.4",
"sequelize-cli": "^2.8.0",
"serve-favicon": "~2.4.3",
"sqlite3": "^3.1.13"
"sqlite3": "^3.1.9"
},
"devDependencies": {
"chai": "^4.1.1",
Expand Down
4 changes: 2 additions & 2 deletions server/src/infrastructure/mailing/jobs-changed-email-template.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ module.exports = {
template += '<ul>';
addedJobs.forEach((job) => {
const formatedDate = moment(job.activity.staffing_needed_from).format('DD/MM/YYYY');
template += `<li><bold>${job.activity.title}</bold> pour le projet ${job.project.name} pour le client ${job.project.customer.name} à partir du ${formatedDate}</li>`;
template += `<li>${job.activity.title} pour le projet ${job.project.name} pour le client ${job.project.customer.name} à partir du ${formatedDate}</li>`;
});
template += '</ul>';
template += '</p>';
Expand All @@ -23,7 +23,7 @@ module.exports = {
template += `${removedJobs.length} mission(s) retirée(s) :`;
template += '<ul>';
removedJobs.forEach((job) => {
template += `<li><bold>${job.activity.title}</bold> pour le projet ${job.project.name}</li>`;
template += `<li>${job.activity.title} pour le projet ${job.project.name}</li>`;
});
template += '</ul>';
template += '</p>';
Expand Down
20 changes: 10 additions & 10 deletions server/test/unit/infrastructure/mailing/jobs-changed-email-template.spec.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -34,14 +34,14 @@ describe('Unit | Infrastructure | Mailing | jobs-changed-email-template', () =>
'<p>Il y a du nouveau du côté du <a href="https://jobs.octo.com">Job Board</a>.</p>' +
'<p>3 nouvelle(s) mission(s) à staffer :' +
'<ul>' +
'<li><bold>job_title_1</bold> pour le projet project_name_1 pour le client customer_name_1 à partir du 10/01/2018</li>' +
'<li><bold>job_title_2</bold> pour le projet project_name_2 pour le client customer_name_2 à partir du 20/02/2018</li>' +
'<li><bold>job_title_3</bold> pour le projet project_name_3 pour le client customer_name_3 à partir du 30/03/2018</li>' +
'<li>job_title_1 pour le projet project_name_1 pour le client customer_name_1 à partir du 10/01/2018</li>' +
'<li>job_title_2 pour le projet project_name_2 pour le client customer_name_2 à partir du 20/02/2018</li>' +
'<li>job_title_3 pour le projet project_name_3 pour le client customer_name_3 à partir du 30/03/2018</li>' +
'</ul></p>' +
'<p>2 mission(s) retirée(s) :' +
'<ul>' +
'<li><bold>job_title_4</bold> pour le projet project_name_4</li>' +
'<li><bold>job_title_5</bold> pour le projet project_name_5</li>' +
'<li>job_title_4 pour le projet project_name_4</li>' +
'<li>job_title_5 pour le projet project_name_5</li>' +
'</ul>' +
'</p>' +
'<p>Pour ne plus recevoir de nouvelles du Job Board, il est possible de <a href="https://jobs.octo.com/#/unsubscribe">se désabonner du Job Board</a>.</p>';
Expand All @@ -63,8 +63,8 @@ describe('Unit | Infrastructure | Mailing | jobs-changed-email-template', () =>
'<p>Il y a du nouveau du côté du <a href="https://jobs.octo.com">Job Board</a>.</p>' +
'<p>2 mission(s) retirée(s) :' +
'<ul>' +
'<li><bold>job_title_4</bold> pour le projet project_name_4</li>' +
'<li><bold>job_title_5</bold> pour le projet project_name_5</li>' +
'<li>job_title_4 pour le projet project_name_4</li>' +
'<li>job_title_5 pour le projet project_name_5</li>' +
'</ul>' +
'</p>' +
'<p>Pour ne plus recevoir de nouvelles du Job Board, il est possible de <a href="https://jobs.octo.com/#/unsubscribe">se désabonner du Job Board</a>.</p>';
Expand All @@ -86,9 +86,9 @@ describe('Unit | Infrastructure | Mailing | jobs-changed-email-template', () =>
'<p>Il y a du nouveau du côté du <a href="https://jobs.octo.com">Job Board</a>.</p>' +
'<p>3 nouvelle(s) mission(s) à staffer :' +
'<ul>' +
'<li><bold>job_title_1</bold> pour le projet project_name_1 pour le client customer_name_1 à partir du 10/01/2018</li>' +
'<li><bold>job_title_2</bold> pour le projet project_name_2 pour le client customer_name_2 à partir du 20/02/2018</li>' +
'<li><bold>job_title_3</bold> pour le projet project_name_3 pour le client customer_name_3 à partir du 30/03/2018</li>' +
'<li>job_title_1 pour le projet project_name_1 pour le client customer_name_1 à partir du 10/01/2018</li>' +
'<li>job_title_2 pour le projet project_name_2 pour le client customer_name_2 à partir du 20/02/2018</li>' +
'<li>job_title_3 pour le projet project_name_3 pour le client customer_name_3 à partir du 30/03/2018</li>' +
'</ul>' +
'</p>' +
'<p>Pour ne plus recevoir de nouvelles du Job Board, il est possible de <a href="https://jobs.octo.com/#/unsubscribe">se désabonner du Job Board</a>.</p>';
Expand Down

0 comments on commit 55948d8

Please sign in to comment.