on the contributors page, only lists the closed PRs and the last date

[deniak]

Fix #270

[Copilot]

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

server.js:695

• The use of the spread operator in 'new Date(...doc.lastUpdated)' assumes that doc.lastUpdated is iterable. If it is a single date value or string, consider using 'new Date(doc.lastUpdated)' to avoid runtime errors.

nonSubstantiveContributors[contributor].prs.push({num: doc.num, lastUpdated: new Date(...doc.lastUpdated).toDateString()});

server.js:702

• Similarly, the use of 'new Date(...doc.lastUpdated)' may be problematic if doc.lastUpdated is not an iterable. Using 'new Date(doc.lastUpdated)' might be more appropriate if doc.lastUpdated is a direct date string or value.

substantiveContributors[affiliationId].prs.push({num: doc.num, lastUpdated: new Date(...doc.lastUpdated).toDateString()});

[dontcallmedom]

LGTM, but yes, I think we should keep track of and distinguish PRs that are merged vs closed (which is part of what #270 is about from my perspective)