old
/
LinkList
2
0
Fork 0
Code Issues 3 Pull Requests Projects Releases Wiki Activity
Labels Milestones

Hookup editor login #23

Merged
a3thakra merged 1 commits from hookup-editor-login into hookup-public-page 2 years ago
Conversation 5 Commits 1 Files Changed 4
a3thakra commented 2 years ago (Migrated from git.uwaterloo.ca)
Owner

cc: @a258wang this is how Steven and I wired up the app to do authentication. Let me know if you have any questions!

cc: @a258wang this is how Steven and I wired up the app to do authentication. Let me know if you have any questions!
q63xu (Migrated from git.uwaterloo.ca) approved these changes 2 years ago
a3thakra commented 2 years ago (Migrated from git.uwaterloo.ca)
Owner

@n3parikh I had to change the scheme from Basic to CustomBasic to prevent popups on 401s.

More info: https://github.com/miguelgrinberg/Flask-HTTPAuth/issues/22

@n3parikh I had to change the `scheme` from `Basic` to `CustomBasic` to prevent popups on 401s. More info: https://github.com/miguelgrinberg/Flask-HTTPAuth/issues/22
a3thakra commented 2 years ago (Migrated from git.uwaterloo.ca)
Owner

loginFailed is a UI concern, not an auth concern.

`loginFailed` is a UI concern, not an auth concern.
a3thakra commented 2 years ago (Migrated from git.uwaterloo.ca)
Owner

We shouldn't setState for components which have been unmounted.

We shouldn't `setState` for components which have been unmounted.
a3thakra commented 2 years ago (Migrated from git.uwaterloo.ca)
Owner

This matches /api/editor/links/foo/bar/whatever

This matches `/api/editor/links/foo/bar/whatever`
q63xu commented 2 years ago (Migrated from git.uwaterloo.ca)
Owner

approved this merge request

approved this merge request

Reviewers

q63xu
The pull request has been merged.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
q63xu
Labels
Apply labels
Clear labels
deployment
No Label deployment
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: old/LinkList#23
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'hookup-editor-login'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes