USACE / USACE/cwms-data-api

Add file-upload field to Ratings

Open
#1,939 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Java
Stars
28
Forks
25
Avg merge
4d 22h
Merged PRs (30d)
20

Description

Is your feature request related to a problem? Please describe.
Given that ratings can be rather large various elements, such as the security scanning elements of a WAF, can behave poorly when excessive data is used.

Additionally various async tools can use used within the application itself to async wait for data to be uploaded and move on to servicing other requests until the full data is ready.

Describe the solution you'd like
Were appropriate on the varius Ratings POST endpoint, allow use of file upload similar to what we are doing for #1718.

Describe alternatives you've considered

Additional context

Contributor guide

Open the contributing guide

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.

Research direction

Start by locating the Ratings POST endpoints and reviewing the file-upload approach described in issue #1718. Determine which endpoints are appropriate for uploaded rating data and how the existing async tools fit the request. Done means the selected Ratings endpoints support file uploads without excessive inline data and are covered by the relevant tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api, backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.