Skip to content

Support document - #168

Merged
sakihet merged 7 commits into
masterfrom
support-document
Aug 4, 2026
Merged

Support document#168
sakihet merged 7 commits into
masterfrom
support-document

Conversation

@sakihet

@sakihet sakihet commented Aug 3, 2026

Copy link
Copy Markdown
Member

No description provided.

@sakihet sakihet self-assigned this Aug 3, 2026

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

This PR updates the migration common library to support Backlog “Document” features by upgrading backlog4j and introducing new domain models, JSON formats, and service APIs to fetch documents, document trees, comments, and attachments.

Changes:

  • Upgrade backlog4j to 2.10.0 and adjust HTTP client initialization accordingly.
  • Add DocumentService (+ implementation) and document-related domain models + JSON marshalling/unmarshalling.
  • Extend project creation and issue filtering to support new/backported backlog4j capabilities (documents, grandchild-related parent/child filtering).

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
core/src/main/scala/com/nulabinc/backlog/migration/common/services/MappingFileService.scala Update Apache Commons CSV CSVFormat construction to builder API.
core/src/main/scala/com/nulabinc/backlog/migration/common/service/ProjectServiceImpl.scala Enable document support when creating projects (documentEnabled).
core/src/main/scala/com/nulabinc/backlog/migration/common/service/IssueServiceImpl.scala Update parentChild filter mapping to new backlog4j parent/child enum values.
core/src/main/scala/com/nulabinc/backlog/migration/common/service/DocumentServiceImpl.scala New service implementation for listing documents, fetching document trees, comments, and attachments.
core/src/main/scala/com/nulabinc/backlog/migration/common/service/DocumentService.scala New service trait defining document-related operations.
core/src/main/scala/com/nulabinc/backlog/migration/common/modules/DefaultModule.scala Wire DocumentService into DI bindings.
core/src/main/scala/com/nulabinc/backlog/migration/common/formatters/BacklogJsonProtocol.scala Add JSON formats for new document domain types and bump BacklogProject format arity.
core/src/main/scala/com/nulabinc/backlog/migration/common/domain/Backlog.scala Add useDocument to BacklogProject and introduce document-related domain case classes.
core/src/main/scala/com/nulabinc/backlog/migration/common/convert/writes/ProjectWithVCSWrites.scala Populate useDocument when converting project data from backlog4j.
core/src/main/scala/com/nulabinc/backlog/migration/common/convert/writes/DocumentWrites.scala New converters from backlog4j document models to internal domain models.
core/src/main/scala/com/nulabinc/backlog/migration/common/convert/BacklogUnmarshaller.scala Add unmarshalling helpers for BacklogDocument and BacklogDocumentTree.
core/src/main/scala/com/nulabinc/backlog/migration/common/conf/BacklogPaths.scala Add filesystem paths for document exports (documents dir, attachments, tree JSON).
core/src/main/scala/com/nulabinc/backlog/migration/common/client/BacklogAPIClientImpl.scala Switch HTTP client implementation to backlog4j’s HttpClientBacklogHttpClient.
build.sbt Upgrade backlog4j version and update scalafix command aliases.

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

@sakihet
sakihet requested a review from yuichi0301 August 4, 2026 07:01

@yuichi0301 yuichi0301 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

@sakihet
sakihet merged commit 73e2a48 into master Aug 4, 2026
1 check passed
@sakihet
sakihet deleted the support-document branch August 4, 2026 08:40
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.

3 participants