Skip to content

CDA-31: Add flag for showing aliased items in catalog/locations#1294

Merged
rma-rripken merged 9 commits into
USACE:developfrom
zack-rma:feature/location_catalog_alias_flag
Sep 9, 2025
Merged

CDA-31: Add flag for showing aliased items in catalog/locations#1294
rma-rripken merged 9 commits into
USACE:developfrom
zack-rma:feature/location_catalog_alias_flag

Conversation

@zack-rma
Copy link
Copy Markdown
Collaborator

@zack-rma zack-rma commented Sep 5, 2025

Fixes #107

Previous PR for TimeSeries alias flag support found here.

Added support for include alias flag for location catalog. Includes integration test.

Comment thread cwms-data-api/src/main/java/cwms/cda/data/dao/LocationsDaoImpl.java Outdated
@zack-rma
Copy link
Copy Markdown
Collaborator Author

zack-rma commented Sep 8, 2025

Failing from CDA-GUI NPM package vulnerabilities.

Comment thread cwms-data-api/src/main/java/cwms/cda/data/dao/LocationsDaoImpl.java Outdated
Comment thread cwms-data-api/src/main/java/cwms/cda/data/dao/LocationsDaoImpl.java Outdated
@zack-rma zack-rma requested a review from rma-bryson September 8, 2025 22:46
Comment thread cwms-data-api/src/test/java/cwms/cda/api/CatalogControllerTestIT.java Outdated
rma-rripken
rma-rripken previously approved these changes Sep 9, 2025
rma-bryson
rma-bryson previously approved these changes Sep 9, 2025
@rma-rripken rma-rripken merged commit 0eb5614 into USACE:develop Sep 9, 2025
8 checks passed
@zack-rma zack-rma deleted the feature/location_catalog_alias_flag branch September 9, 2025 20:25
MikeNeilson pushed a commit that referenced this pull request Oct 23, 2025
Fixes #107

Previous PR for TimeSeries alias flag support found
[here](#1193).

Added support for include alias flag for location catalog. Includes
integration test.

(cherry picked from commit 0eb5614)
MikeNeilson pushed a commit that referenced this pull request Oct 24, 2025
Fixes #107

Previous PR for TimeSeries alias flag support found
[here](#1193).

Added support for include alias flag for location catalog. Includes
integration test.

(cherry picked from commit 0eb5614)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add flag for showing aliased items in catalog/timeseries

3 participants