Skip to content

[Feat] Composer module - #36

Draft
gund wants to merge 118 commits into
masterfrom
feat-composer-module
Draft

gund wants to merge 118 commits into
masterfrom
feat-composer-module

Conversation

@gund

@gund gund commented Mar 15, 2019 •

Copy link
Copy Markdown
Member

@gund gund added type: feature comp: composer Related to Composer package labels Mar 15, 2019
gund added 24 commits March 15, 2019 23:42
And hide internal components DI token
That will return all available components in orchestrator
By replacing Object.values => Object.keys
And use localized imports of modules
And add missing layout module import to unit tests
@gund
gund force-pushed the feat-composer-module branch from e118002 to 19aedc1 Compare June 15, 2019 21:16
gund and others added 19 commits January 15, 2022 00:13
And reset component state when it has to be changed but kept due to tracking optimizations
And render config in a card component with controls in popver
So only one component has it's popover active at a time
import { createMetadataGetSet } from './util';

export interface ConfigurationMeta {
const Reflect = (window as any).Reflect;

@gund gund Jan 15, 2022 •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

User should be able to provide Reflect API differently otherwise default back to getting it from global window.
For example:
https://github.com/orchestratora/gen-io-ts/blob/6f8d6565809100c05fef809fa38942e7086b01d2/projects/gen-io-ts/src/lib/reflect.ts

NOTE: Ideally Reflect API should be shared between libs like @orchestratora/gen-io-ts.

@gund
gund marked this pull request as draft February 6, 2022 22:06
@gund gund changed the title WIP [Feat] Composer module [Feat] Composer module Feb 6, 2022

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp: composer Related to Composer package type: feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant