Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

CHE Factory plugin API #6

Merged
merged 5 commits into from
Dec 19, 2018
Merged

CHE Factory plugin API #6

merged 5 commits into from
Dec 19, 2018

Conversation

vitaliy-guliy
Copy link
Contributor

CHE Factory plugin API Implementation.

@@ -28,7 +28,7 @@ declare module '@eclipse-che/plugin' {
}

export namespace factory {
export function getFactoryById(id: string): PromiseLike<Factory | undefined>;
export function getById(id: string): PromiseLike<MYFactory>;
Copy link
Contributor

Choose a reason for hiding this comment

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

hello, it looks strange to have MyFactory ?

Copy link
Member

Choose a reason for hiding this comment

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

hello, but it is MYFactory but not MyFactory ;)

Copy link
Contributor

Choose a reason for hiding this comment

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

ok @sleshchenko it changes everything 🤣

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed 😆

@vitaliy-guliy vitaliy-guliy merged commit 4997e4f into master Dec 19, 2018
@vitaliy-guliy vitaliy-guliy deleted the che-factory-api-merged branch December 19, 2018 15:35
@benoitf
Copy link
Contributor

benoitf commented Dec 19, 2018

vinokurig pushed a commit that referenced this pull request Apr 6, 2021
Add type of the object to the index file of che-plugin-registry (#6)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants