Skip to content

Sidepanel preview (rebased on master)#258

Open
nwmac wants to merge 34 commits intomasterfrom
sidepanel-preview-master
Open

Sidepanel preview (rebased on master)#258
nwmac wants to merge 34 commits intomasterfrom
sidepanel-preview-master

Conversation

@nwmac
Copy link
Member

@nwmac nwmac commented Jan 10, 2020

No description provided.

Vítor Avelino and others added 30 commits November 29, 2019 14:13
Signed-off-by: Vítor Avelino <vavelino@suse.com>
- Issue 1 (slightly unrelated)
  - app service getspace action used space entity without an org
- Issue 2
  - app entity validation found it was missing the space
  - validation process fetched space with custom action
  - custom action did not contain schema key to use space with org schema
  - org was not stored correctly in store (contained in space rather than seperatly)
- Still to do
  - Fix for issue 2 would need to be expanded to ALL schema's with inline entities
  - These new schemas would need to be added to their entities
  - All usages would have to provide the overriding schemaKey
This reverts commit 0a16203284cc0b42d172972047f5d08cddfecc99.
    - Issue 1 (slightly unrelated)
      - app service getspace action used space schema without an org
    - Issue 2
      - app entity validation found it was missing the space
      - validation process fetched space with custom action
      - custom action did not contain schema key linked to space schema with org
      - this lead to org being not stored correctly in store (contained in space rather than seperatly)
      - Simple fix (see 0a16203284c for harder)
         - When normalizing prioritise the action's schema over attempting
           to fetch via entity catalogue + schemaKey
           (avoids A LOT of plumbing)
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
- Issue 1 (slightly unrelated)
  - app service getspace action used space entity without an org
- Issue 2
  - app entity validation found it was missing the space
  - validation process fetched space with custom action
  - custom action did not contain schema key to use space with org schema
  - org was not stored correctly in store (contained in space rather than seperatly)
- Still to do
  - Fix for issue 2 would need to be expanded to ALL schema's with inline entities
  - These new schemas would need to be added to their entities
  - All usages would have to provide the overriding schemaKey
This reverts commit 0a16203284cc0b42d172972047f5d08cddfecc99.
    - Issue 1 (slightly unrelated)
      - app service getspace action used space schema without an org
    - Issue 2
      - app entity validation found it was missing the space
      - validation process fetched space with custom action
      - custom action did not contain schema key linked to space schema with org
      - this lead to org being not stored correctly in store (contained in space rather than seperatly)
      - Simple fix (see 0a16203284c for harder)
         - When normalizing prioritise the action's schema over attempting
           to fetch via entity catalogue + schemaKey
           (avoids A LOT of plumbing)
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
- Issue 1 (slightly unrelated)
  - app service getspace action used space entity without an org
- Issue 2
  - app entity validation found it was missing the space
  - validation process fetched space with custom action
  - custom action did not contain schema key to use space with org schema
  - org was not stored correctly in store (contained in space rather than seperatly)
- Still to do
  - Fix for issue 2 would need to be expanded to ALL schema's with inline entities
  - These new schemas would need to be added to their entities
  - All usages would have to provide the overriding schemaKey
This reverts commit 0a16203284cc0b42d172972047f5d08cddfecc99.
    - Issue 1 (slightly unrelated)
      - app service getspace action used space schema without an org
    - Issue 2
      - app entity validation found it was missing the space
      - validation process fetched space with custom action
      - custom action did not contain schema key linked to space schema with org
      - this lead to org being not stored correctly in store (contained in space rather than seperatly)
      - Simple fix (see 0a16203284c for harder)
         - When normalizing prioritise the action's schema over attempting
           to fetch via entity catalogue + schemaKey
           (avoids A LOT of plumbing)
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
Signed-off-by: Vítor Avelino <vavelino@suse.com>
@nwmac nwmac self-assigned this Jan 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants