OUPMATH / OxfrodMaths_Prod

0 stars 0 forks source link

Re-sending welcome letter to school admin doen't include host name in prod environemnt #28

Closed mayank2k2 closed 8 years ago

mayank2k2 commented 8 years ago

while want to retrieve the password of School admin. I found that this doesn't contain the host name.

Dear SUMIT MAGO, Your email address has been used to register an account on Oxford Maths You have been registered in: OUP TEST SCHOOL Your role is: School Administrator To start using your account, please sign in at [hostName] with your username and temporary password mentioned below. You need to change the password after signing in. Username: school-admin Password: OUPMath@125 If you have any questions, please email Customer Support at oxfordmathsindia@oup.com. Best wishes, Oxford Maths Team Customer Support

pankaj44 commented 8 years ago

We have investigated the issue and found that we need to replace the hostName in application code .It is small changes there is no relation between issue#24 where we have done the URL redirection for that domain (uat.oxfordmath.co.in) in pigu.html and other places. We will do the desire changes and deployed it in next build (tomorrow build).

mayank2k2 commented 8 years ago

Manas/Pankaj, I want to know how this has broken as this was working before.

Thanks, Mayank Mishra

From: pankaj44 [mailto:notifications@github.com] Sent: Thursday, May 05, 2016 12:50 PM To: manasghosh81/OxfrodMaths_Prod OxfrodMaths_Prod@noreply.github.com Cc: MISHRA, Mayank Mayank.Mishra@oup.com; Author author@noreply.github.com Subject: Re: [manasghosh81/OxfrodMaths_Prod] Re-sending welcome letter to school admin doen't include host name in prod environemnt (#28)

We have investigated the issue and found that we need to replace the hostName in application code .It is small changes there is no relation between issue#24 where we have done the URL redirection for that domain (uat.oxfordmath.co.in) in pigu.html and other places. We will do the desire changes and deployed it in next build (tomorrow build).

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHubhttps://github.com/manasghosh81/OxfrodMaths_Prod/issues/28#issuecomment-217092040

pankaj44 commented 8 years ago

Mayank, The school admin will receive a email after the school registration.The body of the email is based on a standard template which contain a key called [hostname].This key has been replaced from application code by the hostname specified in web-config file.The [hostname] has been used in template to maintain the same template and work the code base seamlessly in different environment

There is a two cases where school admin is getting this email Case 1:- At the time of school registration. Case 2:- Search the school and re-send the email. In case 1 The C# code that sending the email where the key [hostname] is replace by the config host name. Hence once we have changed the config entry with appropriate domain name that changes has been reflected in the email body that school admin is receiving.

But in case 2 of re- send the email for school registration does not contain any line that basically replace the hostname .That's why this error is happening.

This problem is there from beginning.

We have fixed this issue.It will be available in UAT today and Prod tomorrow Regards, Pankaj Kumar.

mayank2k2 commented 8 years ago

Pankaj, First this is not from beginning we used to get welcome letter with proper and this has already tested in UAT. And second why there is 2 different logic/code one for first time welcome letter and another for resend welcome letter

What do you mean ? But in case 2 of re- send the email for school registration does not contain any line that basically replace the hostname .That's why this error is happening.

Thanks, Mayank Mishra

From: pankaj44 [mailto:notifications@github.com] Sent: Thursday, May 05, 2016 3:25 PM To: manasghosh81/OxfrodMaths_Prod OxfrodMaths_Prod@noreply.github.com Cc: MISHRA, Mayank Mayank.Mishra@oup.com; Author author@noreply.github.com Subject: Re: [manasghosh81/OxfrodMaths_Prod] Re-sending welcome letter to school admin doen't include host name in prod environemnt (#28)

Mayank, The school admin will receive a email after the school registration.The body of the email is based on a standard template which contain a key called [hostname].This key has been replaced from application code by the hostname specified in web-config file.The [hostname] has been used in template to maintain the same template and work the code base seamlessly in different environment

There is a two cases where school admin is getting this email Case 1:- At the time of school registration. Case 2:- Search the school and re-send the email. In case 1 The C# code that sending the email where the key [hostname] is replace by the config host name. Hence once we have changed the config entry with appropriate domain name that changes has been reflected in the email body that school admin is receiving.

But in case 2 of re- send the email for school registration does not contain any line that basically replace the hostname .That's why this error is happening.

This problem is there from beginning.

We have fixed this issue.It will be available in UAT today and Prod tomorrow Regards, Pankaj Kumar.

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHubhttps://github.com/manasghosh81/OxfrodMaths_Prod/issues/28#issuecomment-217115482

pankaj44 commented 8 years ago

image

mayank2k2 commented 8 years ago

Pankaj, I like the detail explanation about the issue its cause and resolution and its testing scope mention in github. I hope to keep it for other issue as well so that we should know more about the issue.

@Manas We can discuss the format for final comment when you are marking the issue as Fixed in github.

Thanks, Mayank Mishra

From: pankaj44 [mailto:notifications@github.com] Sent: Thursday, May 05, 2016 3:25 PM To: manasghosh81/OxfrodMaths_Prod OxfrodMaths_Prod@noreply.github.com Cc: MISHRA, Mayank Mayank.Mishra@oup.com; Author author@noreply.github.com Subject: Re: [manasghosh81/OxfrodMaths_Prod] Re-sending welcome letter to school admin doen't include host name in prod environemnt (#28)

Mayank, The school admin will receive a email after the school registration.The body of the email is based on a standard template which contain a key called [hostname].This key has been replaced from application code by the hostname specified in web-config file.The [hostname] has been used in template to maintain the same template and work the code base seamlessly in different environment

There is a two cases where school admin is getting this email Case 1:- At the time of school registration. Case 2:- Search the school and re-send the email. In case 1 The C# code that sending the email where the key [hostname] is replace by the config host name. Hence once we have changed the config entry with appropriate domain name that changes has been reflected in the email body that school admin is receiving.

But in case 2 of re- send the email for school registration does not contain any line that basically replace the hostname .That's why this error is happening.

This problem is there from beginning.

We have fixed this issue.It will be available in UAT today and Prod tomorrow Regards, Pankaj Kumar.

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHubhttps://github.com/manasghosh81/OxfrodMaths_Prod/issues/28#issuecomment-217115482

mayank2k2 commented 8 years ago

Issues is fixed in UAT

manasghosh81 commented 8 years ago

Root case - Code issue there for a particular scenario Resolution - Fixed the code verified scenarios in QC or UAT- Resend email to school admin after search school

mayank2k2 commented 8 years ago

working fine in Prod