Code

HackParser integration#413

Code

Merged

jsmilliken merged 25 commits intomainsourceallies/hacker-rank-queue:mainfrom feature/s3-signed-clean-code-urlssourceallies/hacker-rank-queue:feature/s3-signed-clean-code-urlsCopy head branch name to clipboardOct 21, 2024

+3,665-1,393

Commits

Commits on Oct 4, 2024

• feat/wip: Add basic structure for getting signed S3 URLs

  charlie-map

  committed
  f173479

Commits on Oct 5, 2024

• test: Add PDF file input

  charlie-map

  committed
  5650931

• fix: Revert to older pnpm changes

  charlie-map

  committed
  f8d0da8

• fix: Get the correct lock file version

  charlie-map

  committed
  3a8f236

• fix: Add pdf identifier to test

  charlie-map

  committed
  a61e825

• fix: Remove updated Slack message for tests

  charlie-map

  committed
  761889b

Commits on Oct 18, 2024

• test: Add pdf uploading and fix tests

  

  charlie-map

  authored and

  jsmilliken

  committed
  3ac4318

• test: Add putS3Object mock

  

  charlie-map

  authored and

  jsmilliken

  committed
  70f7af5

• rix: Remove all the test requirements

  

  charlie-map

  authored and

  jsmilliken

  committed
  e5c74fe

• fix: Add example.pdf in app root as well

  

  charlie-map

  authored and

  jsmilliken

  committed
  f7dbf03

• fix: Add more pdfs everywhere

  

  charlie-map

  authored and

  jsmilliken

  committed
  1f1f7a9

• test: Remove everything of practical use

  

  charlie-map

  authored and

  jsmilliken

  committed
  e9edcda

• refactor: Add in slackbot pdfssssssssssssssssssssssssssssssssssss

  

  charlie-map

  authored and

  jsmilliken

  committed
  e23a553

• Update PDF downloading code, added debug log of response

  Show description for 4e084e8
  jsmilliken

  committed
  4e084e8

• Add tests for requesting reviews with a PDF

  Show description for 2b7a870
  jsmilliken

  committed
  2b7a870

• Add PDF & files to updated message on review accept

  Show description for 5aabdcb
  jsmilliken

  committed
  5aabdcb

• Break text up into multiple blocks to avoid 3001 character limit

  jsmilliken

  committed
  00aaa10

• Make HackParser integration togglable

  Show description for a3ad449
  jsmilliken

  committed
  a3ad449

• Cleanup test PDFs, restore coverage thresholds, cleanup comments

  jsmilliken

  committed
  07a8275

• Move HackParser specific logic into HackParserService

  Show description for 1e92cbb
  jsmilliken

  committed
  1e92cbb

• Cleanup tests

  Show description for b1a7dab
  jsmilliken

  committed
  b1a7dab

• Add additional acceptReviewRequest error tests

  jsmilliken

  committed
  e0eb0e6

• Bump line coverage coverage threshold

  jsmilliken

  committed
  5d7d7ca

• Infer type from @smithy/type instead of using it directly

  jsmilliken

  committed
  a7173a3

• Update presigned URL expiration to 7 days

  jsmilliken

  committed
  e27b100