Uploaded image for project: 'Magnolia Synchronization Module'
  1. Magnolia Synchronization Module
  2. MGNLSYNC-43

Finalize support for invoking synchronization asynchronously

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Done
    • Neutral
    • 1.9
    • None
    • None
    • Yes
    • Yes
    • Maintenance 11, Maintenance 12, Maintenance 13
    • 8

    Description

      See the review items on the original ticket:
      https://jira.magnolia-cms.com/browse/MGNLSYNC-40?focusedCommentId=208822&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-208822

      Successfully merged basic functionality and released a 1.9-beta1; Here are additional review items for 2nd-stage review:

      • REST API review - not a problem to change provided we produce a change log / release notes
      • Rename 'repository' params back to 'workspace'
      • Date resolution for the endpoint (help define expected ISO date string format?)
      • Unbox booleans where applicable
      • VH for module config and command bootstraps? non-applicable for new module
      • Move manager to synchronization-core
      • Builder for SynchronizationRequest + pass as param to #synchronize(SyncRequest), #canSynchronize...
      • Remove XML output from endpoint (non-goal)
      • Consider ITs for endpoint?
      • Test method signatures: throw general Exception & no test prefix (BDD-style naming)
      • Failing existing synchronization command tests – failing on JDK12, passing on JDK8
      • Validate module config vs. manager, drop module lifecycle?
      • Reconsider deduplicating `SynchronizationCommand` (preferably evolving or partially deprecating the old one)
      • Global policy? //remove setter on manager, move the configuration under module config

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                rkovarik Roman Kovařík
                slutz Simon Lutz
                Nucleus
                Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Task DoD

                    Time Tracking

                      Estimated:
                      Original Estimate - Not Specified
                      Not Specified
                      Remaining:
                      Remaining Estimate - Not Specified
                      Not Specified
                      Logged:
                      Time Spent - 1d 3h 10m
                      1d 3h 10m