Todo - Githubissues

High priority

Accounts

[ ] Expire passwords that are more than [option] old
[ ] PC invitations
[ ] Merging accounts system-wide
[ ] “Keep me logged in”
[x] Looks like creating a user borrows the password from the contactdb whereas what is really desired is setting the password to empty.
[ ] Clear old passwords
[ ] Looks like “can view reviewer ID“ is not obeyed for mail tool grouping
[ ] Activity log
[ ] Autoassigner: undecided papers is the default
[ ] “Confirm conflict” emails to chair should have link to assign page
[ ] “highlight” in search should show “Did you mean: …”
[ ] “highlight:foo” support
[ ] Configurable conflict types
[ ] Conflict assignment shows asymmetric conflicts
[ ] Conflict assignment shows unlikely conflicts
[ ] Conflict assignment unlikely conflict shows notation
[ ] Lower-case roles
[ ] Report unused CSV columns on user upload
[ ] Security: List sessions
[ ] Security: Log out all sessions
[ ] “# prefs” column on PC view
[x] Security: New style author-view capabilities—do not save in session
[x] revpref API for withdrawn submissions should report a better error, or no error
[ ] Required fields get a red asterisk
[x] Remove review visibility override
[ ] Front page has “open site for reviewing” if reviews are closed but assigned
[x] Sessions with too much data
[ ] Settings search
[ ] Allow admin track permission for default track
[x] Dingbat for zero preferences (gray •?)
[ ] Search term for authors that have more than one submission
[x] Author view capabilities are URL only
[x] Conflictassignment doesn’t render math
[ ] Settings with selectors should have a different UI to allow reordering
Active

Accounts
[ ] “Disable”/“Enable” button on profile page
[ ] Disabled users show up as disableds
[ ] All downloads from users/ page are sorted by Contact::compare
[ ] Track merged contacts in contactdb; report if someone tries to add an old contact
[ ] Tags for all kinds of user
[ ] Conflict matrix / report asymmetric conflicts
Appearance
[ ] i18n: Allow overriding “paper” term w/ “application”
[ ] Review round colors
[ ] Markdown messages
[ ] Better look for infoMsg, etc.; maybe horizontal bars?
[ ] Affordances for j/k/s t etc.
Permissions
[ ] Can authors see the decision? —> Yes, for papers with these tags: ————
Settings
[ ] Settings > Accounts: Remove it
[ ] Settings web site definer
[ ] Settings spacing
Tags
[x] No error message on saving a tag that can't be changed
[x] UI for completion
Search
[ ] Expando for Display options > Formulas
[ ] “Options” for Search > Display options
[ ] Search remembers displayed display options for, e.g., Download>CSV
[ ] show:comments, show:commenters
[ ] topicscore: search keyword
[ ] desirability: search keyword
[ ] plinfo_tags cleanup; allow multiple occurrences per page
[ ] Annotation rows in search say “1 paper (0 shown)”
[ ] Search includes re:me —> "finishreview" paperlinks
[ ] Mouse up/down + x on paper list
[ ] edit:revtype / show:assignment vs. edit:assignment
[ ] Search is changed to apply an override globally & consistently
[ ] topic:>2 etc.
[x] API call for PDF analysis
[x] Collaborators search: Unicode-friendly…?
Documents
[ ] File extensions at download time: use what the user provided, unless it’s a known type like PDF?
[x] Download to filestore: Avoid memory errors for .zip downloads; S3 directly to filesystem
Submissions
[ ] Return on an entry should check override
[ ] Paperstatus JSON save does all permission checks
[ ] PaperStatus: save format
[ ] Submission page: No “ready for review” if final version?
[ ] PaperStatus JSON save can take options not under options
[ ] Submission with override-deadlines fails if submission is not accepted
[ ] “No conflicts” (Andrew Myers)
[ ] Portrait vs. landscape error messages
[ ] Discourage typical bad affiliations
[ ] Collect errors from itext and expose them in a readable manner
[ ] Support document history
[ ] Can change contacts even if not final
[ ] Improve “The submission is ready for review”
[ ] Multiple submission deadlines
Reviews
[ ] empty review terms?
[ ] change where offline review shows up
[ ] requester gets accept notification
[ ] reviewTable: verify $hideUnviewable/can_view_review
[ ] Token reviews don't show up as requested reviews
[ ] Warn on unload if review field is changed
[ ] “Show reviews”/“Show decision” per-paper override
[ ] Show reviews in progress as folded when they are visible at all
[ ] Saving review incorrectly says “no changes” (was missing mandatory field)
[ ] Deleting then re-adding a secondary review screws up reviewNeedsSubmit
[ ] Suggested reviewers for next round (Fred Douglis)
[ ] Review history
[ ] Warn on assigning reviews to expired round
Comments
[ ] Comments can have @-notification
[ ] CommentInfo::save checks permissions
[ ] Disable “Visibility > Hidden from authors” if extrev cannot see comments?
[ ] Comment history
[ ] View plaintext of comment
Tracks
[x] Remove “review visibility override” track
Tracker
[ ] Support PC polls
[ ] Record how much time is spent on each paper
[ ] Tracker notification app?
[x] Multiple trackers
[x] Tracker on mobile
Assignments
[ ] Autoassignment: Balance by explicit weight
[ ] Autoassigner: Ajax posting with polling updates
[ ] Autoassigner: Log parameters
[ ] Autoassigner: Default to undecided papers
[ ] Link up the expertise gadget
[ ] Assigner contacts can create contacts & send email
[ ] New UI for assignments page: button to add a review
[ ] Assigner preference old:new
[ ] Bulk assignment: complain if a field is missing (e.g., tag)
[ ] Bulk assignment: refuse review
[ ] Bulk assignment: remove refuse review
[x] Manual assignment: saveAssignment uses AssignmentSet
[x] Change on autoassign page doesn't highlight “List”
[x] Bulk assign supports tokens
[x] Assigner submit/unsubmit
Graphs
[ ] Wide Y-axis labels increase margin
[ ] Graph size is sensitive to screen size
[ ] Axis control (zooming)
[ ] Graph examples
[ ] Graph library
[ ] CDF with absolute scale not fraction
[ ] More informative tooltips: Y count (CDF)
[ ] Tooltips with “overflow:scroll”
[ ] Search on graph (e.g., PC name)
[ ] Sort X axis by Y axis value
[x] Background color is absolute white
Formulas
[ ] What are the semantics of “count(1)”?
[ ] Displaying review ordinals may be a mistake in that it can help users correlate e.g. preferences with review scores
[ ] ratings
[ ] unparse rounds
Mail
[ ] Manager can only override cc/replyTo for papers they manage?
[ ] Notification emails include opt-out instructions
[ ] Notification emails include opt-out links
[ ] Separate notification options for reviews and comments (Andrei Sabelfeld)
[ ] DKIM
[ ] %DECISION%
[ ] %ABSTRACT%
[ ] How to handle things a recipient may not be allowed to see?
[ ] %IF(papersearch)%
[x] Remove permissionContact
[ ] Add a header to enable bounce tracking
[ ] Track bounces
Action log
[x] CSV download
[x] Edit profile -> action log
[ ] Events: Make sure you sort reviews by reviewAuthorModified when appropriate
[ ] Events: Obey notification
Internals
[ ] Papertable plugins
[ ] HEAD method
[ ] api/paper/NNN/action
[ ] Ajax error responses use non-200-class status
[ ] Render paper strip in JS
[ ] Refactor tag-related JS; remove duplication
[ ] Replace marked with something CommonMark compatible; markdown-it? commonmark.js?
[ ] Plugins to render paper pages
[ ] Searchaction JS
[ ] AssignmentSet uses MessageSet
[ ] whyNot becomes an object/class
[ ] ACM: warn about front matter in secondary sites
[x] Remove PaperColumn::realize
[x] ACM: \frac is parsed in contents
[x] Home plugins
[x] 404 for not-present pages rather than 302
[x] Replace messages.csv (and overrides) with IntlMsgSet
[x] “The submission deadline is NOW” appears broken (maybe only for PC authors)
Uncategorized
[ ] I found myself using DBLP and Google scholar a lot, trying to figure if the reviewer had any paper in common with the reviewers. Would it be possible to automate these searches? I think many PC chairs will be thankful ;)
[ ] Offline reviewing > Tags (John Heidemann)
[ ] Image upload
[ ] JS HTML sanitizer
[ ] CFP upload
[ ] adminhome: Clear message about preferences
[ ] Users list uses the columns framework

Done

Accounts

[x] Enabling accounts should not send email
[x] Should admin be allowed to override paper-manager conflicts via “act as”?
[x] Add test for can_view_contact_tags
Appearance
[x] Links default to not having :visited different
[x] “[L]” and “[1]” look similar, maybe reverse-video the L (Heidemann)
[x] localStorage should use site_key() by default
[x] show:collab shows blank lines poorly
[x] Recent events prevents links on admin panel
[x] Markdown title
[x] Better button appearance
[x] Interleave comments & reviews by timeDisplayed
[x] Markdown text options
[x] Paper option display in PaperTable: no nl2br; use <div class="format0">...
[x] Submission page: Save changes at both top & bottom
[x] Error bubble (e.g. bad pref) has bad tail
[x] Markdown abstract
[x] adminhome: Double-encoding message canbe opted out globally
[x] Home page message stays after you log in
[x] Completion gains keyboard shortcuts: Down to enter completion zone, etc.
[x] Suggestions: Don't filter the list so aggressively
[x] Fix completion in the middle, e.g., “#fo^bar” should highlight “#fobar” if it exists
Permissions
[x] “Administrator” track
[x] Shepherd can be hidden
[x] Overlapping tracks
[x] Allow withdraw after reviews are shown (Eric Bodden)
[x] Problematic display when a chair is both author and reviewer
[x] Disallow external reviewers seeing decisions
[x] Cannot withdraw after reviews are shown
[x] Tracks: “Show all permissions” => “9 permissions have default values / Show all”
Settings
[x] Track settings…Be assigned vs. Self-assign
[x] Reorder track permissions for clarity
[x] Settings > Options JavaScript
[x] Saving new topics appears as “no changes”
[x] Settings > Review form: If error in one field, unfold other modified fields too
[x] Changing an option type may demand option refresh
[x] Settings > Reviews: Update external-review visibility
[x] Reviews visible for XXX tags : responses
[x] “Allow anonymous PC discussion in comments” reword?
[x] Settings > Decisions improvements
[x] Settings time zone selector
[x] Settings allows AoE time zone
[x] “Delete from review form” says whether scores exist
[x] Adding back a field after “Delete from review form” clears existing review values
[x] Settings: Warn on exit with changes
Search
[x] Show contents of a review field
[x] Meeting tracker and conflicts
[x] Action log remembers “act as” actions
[x] PaperSearch: rethink reviewer_user()
[x] override conflicts can affect paper sort order
[x] override conflicts should affect URL
[x] Don't load options by default
[x] Search > Redisplay preserves forceShow
[x] Conflict override propagates to form submission (e.g. download>csv)
[x] Score sort order and conflicts
[x] Collaborators search includes common abbreviations, e.g., UCLA, UCSD, NYU?
[x] show:reviewers lacks preference information
[x] show:revpref sorts by preference
[x] Keyword help: name field values specifically (e.g. “X-Z”, not “worse than X”)
[x] PaperSearch "reviewer" param
[x] Does reviewerUser and t: rable expose information to PC members?
[x] Ambiguous abbreviations (e.g., review field AND option): fix suggestions
[x] ReviewDelegationPaperColumn and multiple rows
[x] Test ire:, pre:
[x] Test -re:
[x] Columns for Search > Display options
[x] Move “Edit formulas” up near formulas
[x] Score sort order is displayed immediately via ajax
[x] JSON-extensible search keywords
[x] Move review fields, options into JSON framework
[x] Extensible search
[x] api/search action
[x] Hook up api/search to sorting
[x] List cookies: compress matchPreg representation
[x] Focus is on preferences if focus was on preferences
[x] Statistics are on by default
[x] Statistics are updated on show folded field
[x] Fix that missing pidfield nonsense
[x] Fix nonsense with pldisplay and new review field names
[x] Remove redundant “url” hotlist argument
[x] Options and scores use same abbreviation/search-keyword stuff
[x] Paper option column display: Documents
[x] Use delegation for paper list actions in case of slow load
Documents
[x] Download > Final versions —> no final version —> bad error message
Submissions
[x] Upload —> Remove
[x] Collaborators/Collaborators conflict detection on manual assignment
[x] Submission page: Highlight if authors are anonymous
[x] Document upload doesn't save as a duplicate if filename is different
[x] Submission page: Don't show full edit if conflicted chair
[x] Submission page: “Contacts” is included even on initial entry
[x] Submission page: Button to replace PDF?
[x] Log what changed
[x] Submission page: Warn if leave page with pending changes
[x] Submission page: Conflicts are highlighted with a check mark or something
[x] Submission page: Preview for abstract
[x] If you submit a final version, it needs to set timeSubmitted
[x] Add a test that rejected papers cannot be updated
[x] “Attachments” options default to “Prominent”
[x] “Default” visibility is default—defaults to “Prominent” for attachments/supplements, under “topics” otherwise
[x] Saving paper with a contact with a bad email: Allow it
[x] Collaborators are folded along with Authors on paper pages
Reviews
[x] If no self-assignment, do not show review form to chair
[x] if author will get notified, warn reviewer
[x] list refusals with “delete refusal” buttons
[x] “Note for reviewer (optional)” only appears if %REASON% is in the email template
[x] Allow requesters to preview email to be sent to new reviewer
[x] Copy from delegated review
[x] Cancel button
[x] Like/dislike below review
[x] Remove/deprecate “current round” selector in favor of “lowest unexpired round”
[x] Check round selector has reasonable default
[x] Problems with round display order, empty rounds being shown
[x] store review texts in a separate JSON
[x] Log what changed
[x] Markdown-friendly pretty text form
[x] Don't change reviewModified if review fields not modified
[x] Hide a review from users if there's no user-visible information in it
[x] Review > Preview
[x] reviewWords counts visible fields
[x] reviewWords: track when the review form changed
[x] Adjust offline review formatting for Choices and visibility
[x] Save review —> list becomes Submissions not Your reviews
[x] Review delegation delegates the reviewer’s round
Comments
[x] Visual appearance for errors, warnings
[x] Rebuttals: Tell people about notification
[x] Comments can have attachments
[x] Comment editor: Tag is a button
[x] Comment editor: “Delete comment” button changes
[x] Remove #response tags from downloaded comments
[x] Open responses for subsets of papers
[x] Meta-Return submits
[x] Authors can leave comments
[x] Preview Response shows over-word-limit bar
[x] Comments have a blob if they get too long
Tracker
[x] On 502 from trackercomet, reduce timeout from ~5m
[x] Comet tracker problems with multiple tabs
[x] track when the tracker was most recently turned off
Assignments
[x] Warn on navigation if assignments not saved
[x] Preserve badpairs if nothing to assign
[x] Does bulk-assigning external reviewers work?
[x] Assigner contacts
[x] Assigner decision
[x] “Assignments” on autoassigner listed as “Changes”
[x] Show preferences when assigning conflicts
[x] Bad pairs on autoassigner: send only those defined
[x] Assignment summary: Make an appropriate summary for the assignment type
[x] Autoassignment PC list: Columns
[x] Assigner: Checkboxes on assignment listing
Graphs
[x] Show mean on boxplots
[x] “# papers” axis label is wrong for by-review graphs
[x] On CDF by letter score, the scale is backwards
[x] More informative tooltips: X value, Y value
Formulas
[x] display “reviewer”
[x] “reviewer:LIMIT” works like “re:LIMIT”
[x] “re:reviewer” obeys visibility of this review’s identity
[x] “opt:WHATEVER” can return numeric
[x] “reviewer” cannot be used in math expressions
[x] “[score]:external”, “external:[score]”
[x] reference a formula by name in another formula
[x] # authors formula
Mail
[x] Mail including names is never combined
Action log
[x] Consolidate mail entries
[x] Blind administered conflicts
[x] Filtered version available to administrators
[x] Consolidate tag log entries
Internals
[x] Remove Conf::reviewRow in favor of functions on PaperInfo
[x] review_rows: remove it, since reviews should be ReviewInfo; fix activity to load papers separately
[x] Drop main textual review fields
[x] Drop main textual review fields from DB
[x] PaperSearch always uses PaperInfo methods, not special-purpose crap loaded from the DB
[x] No inline on handlers
[x] Conf::topic_map() uses stupidity
[x] Contact::is_password_disabled(): remove in favor of is_disabled()
- [x] assertion that if password==="", !!disabled
- [x] change databases using a schema upgrade
[x] passwordIsCdb issues
[x] Dbl::qe() should error_log as well as Conf::error()
[x] More primary keys
[x] Move search page to new-style ajax from old-style ajax
[x] New-style ajax: on timeout/other error, call callback
[x] Plugins to render paper list footer
[x] Render paper reviews entirely in JS
[x] Paper list: the id is an argument

kohler / hotcrp

Todo #69

High priority

Accounts

Active

Accounts

Appearance

Permissions

Settings

Tags

Search

Documents

Submissions

Reviews

Comments

Tracks

Tracker

Assignments

Graphs

Formulas

Mail

Action log

Internals

Uncategorized

Done

Accounts

Appearance

Permissions

Settings

Search

Documents

Submissions

Reviews

Comments

Tracker

Assignments

Graphs

Formulas

Mail

Action log

Internals