Skip to content

Follow-up items from PR #2390 review — Draft APIs error handling, cleanup, and tests #2402

Description

@Varaniya201

Description

This issue tracks the outstanding items identified during the review of PR #2390 that need to be addressed before/after merge. These include a blocker bug fix, a performance concern, error-handling design decisions for the new draft-related APIs, missing unit tests, and a minor formatting fix.

Tasks

  • Blocker bug ([review comment (https://github.com/github-issue#2326, #2385 #2390#discussion_r3878659189))
  • Remove the MVEL code flow as it is no longer being used.
  • Error handling design for draft-related APIs
  • Unit test cases
  • Add unit test coverage for the changes introduced in this PR.
  • Performance fix (review comment) : Address the performance issue flagged in the review comment.
  • Formatting fix — "anonymous" tag casing: Update the anonymous tag to use a capitalized format, as the current lowercase value is inconsistent with the existing tag naming convention.

Reference PR

#2390

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

Fields

Priority

None yet

End Date

None yet

Complexity

None yet

Start Date

None yet

Original Estimate

None yet

Time Tracking

None yet

Resolved By

None yet

Story Points

None yet

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions