Uploaded image for project: 'Magnolia REST Framework'
  1. Magnolia REST Framework
  2. MGNLREST-133

Check included node-types in query responses

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Neutral
    • 2.0
    • None
    • None
    • None

    Description

      • It seems confusing that nodeTypes config may be used for checking node types of "top-level" items only, and could still include "unwanted" node-types as child-nodes
      • corollary is that if we want to include some child-nodes, then e.g. blocks may be returned in the same result set as stories, as top-level elements (despite being maybe included under stories as well). This mixture of types on root-level would be a bit more difficult to process on client side.

      We first verify whether these statements are correct.

      Then we should probably consider splitting nodeTypes config into primaryNodeTypes vs. secondaryNodeTypes (or maybe some other property-based flavor of config).

      With this, developers would get more clarity about what to expect in responses.

      See also
      https://wiki.magnolia-cms.com/display/TH2/REST+TEST+Topher+Notes#RESTTESTTopherNotes-ResultsetIssues

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              sang.ngo Sang Ngo Huu
              mgeljic Mikaël Geljić
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Bug DoR
                  Task DoD

                  Time Tracking

                    Estimated:
                    Original Estimate - 2d Original Estimate - 2d
                    2d
                    Remaining:
                    Remaining Estimate - 0d
                    0d
                    Logged:
                    Time Spent - 3d 2.5h
                    3d 2.5h