Skip to content

Conversation

e-hosea
Copy link

@e-hosea e-hosea commented Sep 2, 2025

What does this PR do?

This PR exports the rest endpoints so that it is usable in ddf-ui

Who is reviewing it?

Select relevant component teams:

Ask 2 committers to review/merge the PR and tag them here.

How should this be tested?

Any background context you want to provide?

What are the relevant tickets?

Fixes: #____

Screenshots

Checklist:

  • Documentation Updated
  • Update / Add Threat Dragon models
  • Update / Add Unit Tests
  • Update / Add Integration Tests

Notes on Review Process

Please see Notes on Review Process for further guidance on requirements for merging and abbreviated reviews.

Review Comment Legend:

  • ✏️ (Pencil) This comment is a nitpick or style suggestion, no action required for approval. This comment should provide a suggestion either as an in line code snippet or a gist.
  • ❓ (Question Mark) This comment is to gain a clearer understanding of design or code choices, clarification is required but action may not be necessary for approval.
  • ❗ (Exclamation Mark) This comment is critical and requires clarification or action before approval.

@e-hosea e-hosea requested a review from stustison as a code owner September 2, 2025 04:25
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Hosea, Elizabeth K. [AU-AU] seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@jlcsmith
Copy link
Member

jlcsmith commented Sep 2, 2025

@e-hosea - did you intend to create this PR against 2.29.x instead of master?

@e-hosea
Copy link
Author

e-hosea commented Sep 3, 2025

@e-hosea - did you intend to create this PR against 2.29.x instead of master?

@jlcsmith Prob both

@e-hosea e-hosea closed this Sep 16, 2025
@e-hosea
Copy link
Author

e-hosea commented Sep 16, 2025

no longer needed, using a different approach

@e-hosea e-hosea deleted the export-rest-endpoints branch September 16, 2025 01:04
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.

4 participants