Skip to content
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

Don't allow uploading a previously-uploaded puzzle? #139

Open
dpsanders opened this issue Jan 2, 2021 · 3 comments
Open

Don't allow uploading a previously-uploaded puzzle? #139

dpsanders opened this issue Jan 2, 2021 · 3 comments

Comments

@dpsanders
Copy link

Maybe it would be a good idea to not allow a puzzle to be uploaded if it has previously already been uploaded?

@ewmson
Copy link
Contributor

ewmson commented Jan 25, 2021

Now that the puzzles have been moved into postgres we should be able to enforce this with a constraint.

I think we want to allow non public (people testing or not wanting to publish) to have duplicates and only apply the constraint on the public one.

If we do go this route, there is a question of the past data as solves will be associated with different puzzle ids, but refer to the exact same puzzle. I suspect we can just start enforcing the constraint going forward to not worry about the past.

@fg123
Copy link

fg123 commented Nov 16, 2021

In a similar realm, including other types of meta-data would be useful too, such as an "uploaded date" versus the puzzle date (i.e. for dated puzzles like NYT or LA Times). This would allow for retroactive uploading of puzzles but still allow front-end to sort by puzzle date, for example.

@stevenhao
Copy link
Member

duplicate of #15 (no pun intended)

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

No branches or pull requests

4 participants