Skip to content

chore: Refactor ADEX request visualization to use UID#22990

Open
jason-p-pickering wants to merge 4 commits intomasterfrom
uid-adex-requests-visualization
Open

chore: Refactor ADEX request visualization to use UID#22990
jason-p-pickering wants to merge 4 commits intomasterfrom
uid-adex-requests-visualization

Conversation

@jason-p-pickering
Copy link
Contributor

@jason-p-pickering jason-p-pickering commented Feb 18, 2026

Refactor String to UID for visualizations in an aggregate data exchange.

Worth noting that the response with invalid UIDs is different and this may have implications on the front end:

Before:

  {                                                                                                                                            
    "httpStatus": "Conflict",                                                                                                                  
    "httpStatusCode": 409,                                                                                                                     
    "response": {
      "responseType": "ImportReport",
      "errorReports": [{ "errorCode": "E4014", "message": "..." }]
    }
  }

After:

  {
    "httpStatus": "Conflict",
    "httpStatusCode": 409,
    "message": "UID must be an alphanumeric string of 11 characters... (through reference chain: AggregateDataExchange->Source->...)"
  }

Since the front-end uses a picker to choose the visualization, this is not in practice expected to be a problem.

Limitations: Does not validate that the visualization actually exists, which is the current behavior.

@jason-p-pickering jason-p-pickering requested a review from a team February 18, 2026 17:35
@jason-p-pickering jason-p-pickering marked this pull request as draft February 18, 2026 17:56
@jason-p-pickering jason-p-pickering marked this pull request as ready for review February 18, 2026 19:33
@sonarqubecloud
Copy link

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.

2 participants

Comments