Uploaded image for project: 'Magnolia UI'
  1. Magnolia UI
  2. MGNLUI-5945

Omit the most typical app/sub-app class references in configuration

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Neutral Neutral
    • None
    • None
    • None

      We still need to specify the usually same sub-app/app classes, can we fallback to defaults there? Aliases could be introduced to simplify this too.

      Examples.

      Typical content app def/app class:
      class: info.magnolia.ui.contentapp.configuration.ContentAppDescriptor
      appClass: info.magnolia.ui.framework.app.BaseApp
      
      Browser sub-app
        browser:
          class: info.magnolia.ui.contentapp.configuration.BrowserDescriptor
      
      Detail sub-app:
          class: info.magnolia.ui.contentapp.detail.DetailDescriptor
      

      Those show up in 90% (if not more) app descriptors and are usually the same.

      Acceptance criteria

      Put also to

      • personas
      • segments
      • pages
      • assets
      • definitions
      • resources

      Discussion points

      These aliases would not be useful probably because the content type app already has it preconfigured and you would use it only in custom apps which are written by Java developers anyway, who would benefit from using the class names more than arbitrary aliases. These could even introduce inconsistency in configuration.

      We were motivated because we saw it in our own defs, but we'd better start using content-type apps with decorations instead.

      PM input: We still have a lot of users who write content apps not using content type app generation, so this would still be beneficial

        Acceptance criteria

              Unassigned Unassigned
              sdemocko Šimon Demočko
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:

                  Task DoD