Closed kudhru closed 7 years ago
Are we integrating with django app ?. In that case we can add @login-required
to these pages
Siddharth will let you know about everything in the future.
On 16 Mar 2017, 10:31 +0530, Partho Sarthi notifications@github.com, wrote:
Are we integrating with django app ?. In that case we can add @login-required to these pages
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub (https://github.com/parthosa/arcd-transcripts/issues/14#issuecomment-286958580), or mute the thread (https://github.com/notifications/unsubscribe-auth/AMpf2qwSlUfMeSB3JX2FAlphcqZw9-Edks5rmMIegaJpZM4MP31t).
Ok as of now i have added the required page redirections.
Don't show the pages without user being logged in, which require user to be logged in. For example, request transcript page is currently visible when the user is not logged in. In such cases, redirect the user to login page.