@shailaja I tested the latest code in shailja_test branch on test2 server and found that it's working fine, meaning url is generated properly. Please update this PR to include latest changes.
@shailaja I looked at the image and the latest code on the test server. The code on the test server does not match with the one in the PR. Could you test once on the updated code? Let me know if…
@shailaja changes looks fine now. Have you tested the app code with the latest changes?
yes ,but facing the same issue which is because of email
That's not a problem. That's…
Format this part. Does not look very nice. Maybe pass each parameter on a different line.
@shailaja changes looks fine now. Have you tested the app code with the latest changes?
@shailaja Please include only the required changes.
Upon comparision I found that
- Entire 'home' function has been changed, which is not reuired. Ideally, you'd only add the section to create…
Just saw that new html templates have been added in the outer 'templates' folder instead of 'app/templates' folder where it's actually being referenced from. I think we should also remove outer…
A simple href with already constructed url looks a good suggestion to me.