-
Notifications
You must be signed in to change notification settings - Fork 0
feat: adds user authentication #218
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 6 commits
da76704
57b3394
96b2fb7
6e61d16
1449b87
9862077
89c5724
060c9cd
0a709a3
05123c4
42fc55f
a95734d
e20efd1
2682be5
9e8c9f9
ca2b86b
d24c537
01cf8ed
816f7db
e80104d
2f1f5dd
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,5 +2,7 @@ | |
exclude = | ||
.git, | ||
__pycache__, | ||
build | ||
build, | ||
venv, | ||
.venv | ||
max-complexity = 10 |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -49,7 +49,8 @@ | |
<a href="/job_params">Job Parameters</a> | | ||
<a title="Download job template as .xslx" href= "/api/job_upload_template" download>Job Submit Template</a> | | ||
<a title="List of project names" href= "{{ project_names_url }}" target="_blank" >Project Names</a> | | ||
<a title="For more information click here" href="https://aind-data-transfer-service.readthedocs.io" target="_blank" >Help</a> | ||
<a title="For more information click here" href="https://aind-data-transfer-service.readthedocs.io" target="_blank" >Help</a> | | ||
<a href="/login">Sign-In</a> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you add the Sign-In button to the navbar in the other pages? Namely, the Job Status and Job Params pages. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I am not entirely sure if the Sign-In button is necessary, as it's kind of doesn't add anything. I am actually thinking to remove it. @jtyoung84? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Replaced with button for Admin page instead |
||
</nav> | ||
<br> | ||
<div> | ||
|
Uh oh!
There was an error while loading. Please reload this page.