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

Zero-input route to new, blank Document #1226

Merged
merged 5 commits into from
Nov 29, 2023
Merged

Conversation

jvwong
Copy link
Member

@jvwong jvwong commented Nov 27, 2023

Created a new route /document/new that will create a new, blank Document with article and correspondence information stubbed out (so it is flagged as missing in admin). Updated robots.txt to get this route disallowed.

Case: No author information provided

Case: Author information provided

Admin view
Screenshot 2023-11-27 at 4 06 59 PM

Refs #1056

@jvwong jvwong requested a review from maxkfranz November 27, 2023 21:17
Copy link
Member

@maxkfranz maxkfranz left a comment

Choose a reason for hiding this comment

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

Looks good. The changes don't seem invasive.

@jvwong jvwong merged commit a2916cd into unstable Nov 29, 2023
@jvwong jvwong deleted the iss1056_get-document branch November 29, 2023 14:50
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