Skip to content

Comments

Allow server to provide aggregate relations#17

Merged
codeliner merged 1 commit intomasterfrom
feature/server_aggregate_relations
Sep 20, 2020
Merged

Allow server to provide aggregate relations#17
codeliner merged 1 commit intomasterfrom
feature/server_aggregate_relations

Conversation

@codeliner
Copy link

With the PR the server can provide aggregate relations that are merged with the user defined aggregateConfig. This feature is particularly useful for the integration with InspectIO and the code generator. Developers can define relations already in InspectIO and the info is forwarded by the backend to Cockpit.

Copy link
Collaborator

@martin-schilling martin-schilling left a comment

Choose a reason for hiding this comment

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

Nice Feature 😃 We‘ll need to think about documentation later since there are three ways (config file, server and hooks) to configure this now which is a little complicated

@codeliner
Copy link
Author

@martin-schilling yeah, that's true!

InspectIO integration was the more important factor for me here. Goal is to let InspectIO and Cockpit play really nice together with zero configuration for the user. But both tools should still work standalone, hence we need the other options, too.

By the way, I have working Node.JS TypeScript backend connected to Cockpit, with 90% of the code generated from an InspectIO event map ;) Can't wait to show you this baby in October.

@codeliner codeliner merged commit 06f23f7 into master Sep 20, 2020
@codeliner codeliner deleted the feature/server_aggregate_relations branch September 20, 2020 20:50
@martin-schilling
Copy link
Collaborator

By the way, I have working Node.JS TypeScript backend connected to Cockpit, with 90% of the code generated from an InspectIO event map ;) Can't wait to show you this baby in October.

Sounds great! Can‘t wait to check it out

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants