Skip to content

Ensure ComputationRefs endpoint only returns refs, not computations#388

Merged
adamkorynta merged 2 commits intoopendcs:mainfrom
zack-rma:bugfix/computation_refs
Mar 27, 2025
Merged

Ensure ComputationRefs endpoint only returns refs, not computations#388
adamkorynta merged 2 commits intoopendcs:mainfrom
zack-rma:bugfix/computation_refs

Conversation

@zack-rma
Copy link
Collaborator

Problem Description

Fixes #383.

GET /computationrefs is returning full computation DTO's

Solution

Replaced method call used in computation reference retrieval. Updated integration testing to match.

how you tested the change

Integration tested against OpenTSDB and CWMS database instances.

Where the following done:

  • Tests. Check all that apply:
    • Unit tests created or modified that run during ant test.
    • Integration tests created or modified that run during integration testing
      (Formerly called regression tests.)
    • Test procedure descriptions for manual testing
  • Was relevant documentation updated?
  • Were relevant config element (e.g. XML data) updated as appropriate

If you aren't sure leave unchecked and we will help guide you to want needs changing where.

@zack-rma zack-rma requested a review from adamkorynta March 26, 2025 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

CWMS Computations tab shows DataTables error

2 participants