Skip to content

Archive explorer, removal of <CANDIDATE>#9

Open
danielmawhirter wants to merge 4 commits intomainfrom
archive-explorer
Open

Archive explorer, removal of <CANDIDATE>#9
danielmawhirter wants to merge 4 commits intomainfrom
archive-explorer

Conversation

@danielmawhirter
Copy link
Copy Markdown
Contributor

No description provided.

@danielmawhirter danielmawhirter requested a review from ddn0 April 21, 2021 21:55
@danielmawhirter
Copy link
Copy Markdown
Contributor Author

Screen Shot 2021-04-21 at 4 58 03 PM

<body onload="init()">
<div id="ref_list"></div>
<script>
function initList() {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was actually thinking that we make past interviews harder to find rather than easier.

With this change it would technically be possible for a candidate to find old interviews more easily (if they could guess /archive as an interesting path) if I'm understanding it correctly.

Copy link
Copy Markdown
Contributor Author

@danielmawhirter danielmawhirter Apr 22, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, understood, this is primarily a permission issue then. Here's what I'd like to do:
After a document is first created, anyone who knows the link can read or write it for 2? days. After that, you have to sign in with a Katana login (firebase can do it), then you can read anything (maybe not write).
I'm looking at some documentation to figure out how to do this, does it seem reasonable?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That sounds great!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants