Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

Include local time in downloaded report filenames

Closed
  • Is something wrong?
#309 1 comment 0 reactions 1 assignee View on GitHub

Maintainers usually reply within 1 day

@106081011 is already working on this.

Since Oct 14, 2026.

  • #313 by @106081011 — merged

Assessment

This issue has not been assessed yet.

Description

Current behavior

reportFilename() in web/download.js generates filenames using only the problem ID and local calendar date.

When multiple interviews for the same problem occur on the same day, their reports receive the same suggested filename, making them difficult to distinguish by name.

Expected behavior

Include the local time in the filename so reports from different interview times on the same day are easier to identify.

For example:

  • Current: interview-report-two-sum-2026-10-14.md
  • Proposed: interview-report-two-sum-2026-10-14-153000.md

The proposed suffix uses the format YYYY-MM-DD-HHMMSS.

Proposed change

Update reportFilename() to include the local hours, minutes, and seconds while preserving the existing problem ID sanitization and date formatting.

Update tests/browser/download.test.js to verify the new format, including local date boundaries and different interview times on the same day.

This change only affects downloaded filenames and does not modify report contents.

Dominant language
Rust
Stars
160
Forks
64
Avg merge
1d 11h
Merged PRs (30d)
132

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from sysprog21/codetrial

All issues in sysprog21/codetrial

Similar issues

More Rust issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.