Skip to content

chore(retentions): add new retentions method for support drafts#7

Open
raul-facturapi wants to merge 1 commit into
mainfrom
chore/retention_draft
Open

chore(retentions): add new retentions method for support drafts#7
raul-facturapi wants to merge 1 commit into
mainfrom
chore/retention_draft

Conversation

@raul-facturapi

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds draft-related Retentions APIs to the Java SDK and validates their HTTP behavior via resource tests, alongside a patch-version bump and changelog entry for the new surface area.

Changes:

  • Add draft support methods to RetentionsResource (updateDraft, copyToDraft, stampDraft) and a convenience cancel(String id) overload.
  • Add unit tests asserting correct HTTP methods, paths, and request bodies for retention draft flows.
  • Bump SDK version to 1.3.1 and document the changes in CHANGELOG.md (and update README dependency snippets).

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/test/java/io/facturapi/FacturapiResourcesTest.java Adds coverage for retention draft create/update/copy/stamp/cancel/list request construction.
src/main/java/io/facturapi/resources/RetentionsResource.java Introduces draft-oriented retention endpoints and a convenience cancel overload.
README.md Updates dependency examples to version 1.3.1.
README.es.md Updates dependency examples to version 1.3.1.
pom.xml Bumps project version from 1.3.0 to 1.3.1.
CHANGELOG.md Adds 1.3.1 entry describing the new retentions draft methods.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/main/java/io/facturapi/resources/RetentionsResource.java
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