Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat: Visual canvas dashboard #6294

Open
wants to merge 65 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
65 commits
Select commit Hold shift + click to select a range
002a5f9
Remove data resolvers from components
begelundmuller Oct 25, 2024
e873f1e
wip
briangregoryholmes Nov 5, 2024
2b80cff
merge main
briangregoryholmes Nov 5, 2024
0ffe720
wip
briangregoryholmes Nov 7, 2024
27b579f
Merge branch 'main' into bgh/visual-explore-editing
briangregoryholmes Nov 7, 2024
05e361e
create theme input
briangregoryholmes Nov 7, 2024
27729ea
add default case when key not set
briangregoryholmes Nov 7, 2024
add21fd
cleanup
briangregoryholmes Nov 7, 2024
36c1a44
merge main
briangregoryholmes Nov 7, 2024
0c2d217
Merge branch 'main' into bgh/visual-explore-editing
briangregoryholmes Nov 8, 2024
96c827f
cleanup and set defaults feature
briangregoryholmes Nov 8, 2024
a6ce459
test fix
briangregoryholmes Nov 9, 2024
595ffbb
Init canvas screen
djbarnwal Nov 14, 2024
4d25035
uxqa
briangregoryholmes Nov 15, 2024
2a972b2
Merge branch 'main' into bgh/visual-explore-editing
briangregoryholmes Nov 15, 2024
c62f476
uxqa
briangregoryholmes Nov 15, 2024
e17b46d
add tabs for chart options
djbarnwal Nov 18, 2024
8ac6ccb
Working chart
djbarnwal Nov 18, 2024
d610d84
uxqa, exclude mode
briangregoryholmes Nov 18, 2024
0ed000b
Merge branch 'main' into feat/canvas-2.0
djbarnwal Nov 19, 2024
065bc28
Merge branch 'bgh/visual-explore-editing' into feat/canvas-2.0
djbarnwal Nov 19, 2024
a25becc
Update UI to match visual editing
djbarnwal Nov 19, 2024
a289a2f
Remove unused file
djbarnwal Nov 19, 2024
76835a6
Larger icons
djbarnwal Nov 19, 2024
1b356b9
merge main
djbarnwal Nov 24, 2024
e8ca858
Merge branch 'main' into feat/canvas-2.0
djbarnwal Nov 25, 2024
33fa567
Share props
djbarnwal Nov 25, 2024
0512cab
Merge branch 'main' into begelundmuller/remove-component-resolvers
begelundmuller Dec 4, 2024
9ffddc4
Fix test
begelundmuller Dec 4, 2024
a2a4d7f
Add `max_width` and `theme`; remove `gap` and `column`; move `vega_li…
begelundmuller Dec 4, 2024
3434bc0
Add state and visual inspector
djbarnwal Dec 5, 2024
fe72b44
Merge main
djbarnwal Dec 5, 2024
2faedd8
Remove logs
djbarnwal Dec 5, 2024
f4d976b
Add component framework
djbarnwal Dec 10, 2024
97f1b5a
Merge main
djbarnwal Dec 10, 2024
d91c5d3
Fallback for MesaureBigNum for canvas
djbarnwal Dec 10, 2024
f6a3199
Merge branch 'main' into feat/canvas-2.0
djbarnwal Dec 12, 2024
5c548e8
Add input UI
djbarnwal Dec 14, 2024
44b9013
Merge main
djbarnwal Dec 14, 2024
b99aebe
Update schema
djbarnwal Dec 17, 2024
453a46d
Merge branch 'main' into feat/canvas-2.0
djbarnwal Dec 17, 2024
bec6332
Add bar positional inputs
djbarnwal Dec 17, 2024
ee35a13
Remove old import
djbarnwal Dec 18, 2024
4476e5a
Merge main
djbarnwal Dec 18, 2024
6a998c1
Merge branch 'main' into begelundmuller/remove-component-resolvers
begelundmuller Dec 18, 2024
26cc8e3
Remove non required fields from spec
djbarnwal Dec 18, 2024
871b1a8
Merge branch 'begelundmuller/remove-component-resolvers' into feat/ca…
djbarnwal Dec 18, 2024
f237dde
Remove old canvas deprecated properties
djbarnwal Dec 18, 2024
4458b80
Code hygiene
djbarnwal Dec 18, 2024
6e3337b
Remove eslint errors
djbarnwal Dec 19, 2024
fd4020f
Relax types for updating property
djbarnwal Dec 19, 2024
0bb083f
Remove stale code
djbarnwal Dec 19, 2024
70a22a3
Update preview
djbarnwal Dec 19, 2024
40d5875
Merge main
djbarnwal Dec 19, 2024
eb5bcb9
Update file artifact references
djbarnwal Dec 20, 2024
41c2d12
Remove unused prop
djbarnwal Dec 20, 2024
dfe6d62
remove ai chart gen from codebase
djbarnwal Dec 20, 2024
302ca97
Add support for adding component from menu
djbarnwal Dec 20, 2024
f563a09
Fix Go lint and test
begelundmuller Dec 20, 2024
b656cff
Use first metricsView as default
djbarnwal Dec 20, 2024
223d203
Merge remote-tracking branch 'origin/feat/canvas-2.0' into feat/canva…
djbarnwal Dec 20, 2024
ba704d0
Code hygiene
djbarnwal Dec 20, 2024
001b84c
use query for valid spec store
djbarnwal Dec 20, 2024
edd973d
Wait for container dimensions to be available before render
djbarnwal Dec 20, 2024
bab44b4
Fix fileartifact reference in component class
djbarnwal Dec 20, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
353 changes: 126 additions & 227 deletions package-lock.json

Large diffs are not rendered by default.

1,758 changes: 866 additions & 892 deletions proto/gen/rill/runtime/v1/queries.pb.go

Large diffs are not rendered by default.

63 changes: 0 additions & 63 deletions proto/gen/rill/runtime/v1/queries.pb.validate.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

530 changes: 260 additions & 270 deletions proto/gen/rill/runtime/v1/resources.pb.go

Large diffs are not rendered by default.

64 changes: 31 additions & 33 deletions proto/gen/rill/runtime/v1/resources.pb.validate.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

23 changes: 6 additions & 17 deletions proto/gen/rill/runtime/v1/runtime.swagger.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -1133,7 +1133,7 @@ paths:
properties:
args:
type: object
title: Optional args to pass to the data resolver and for resolving templating in the renderer properties
title: Optional args for resolving templating in the renderer properties
tags:
- QueryService
/v1/instances/{instanceId}/queries/descriptive-statistics/tables/{tableName}:
Expand Down Expand Up @@ -3471,12 +3471,13 @@ definitions:
properties:
displayName:
type: string
columns:
type: integer
format: int64
gap:
maxWidth:
type: integer
format: int64
theme:
type: string
embeddedTheme:
$ref: '#/definitions/v1ThemeSpec'
variables:
type: array
items:
Expand Down Expand Up @@ -3826,10 +3827,6 @@ definitions:
type: string
description:
type: string
resolver:
type: string
resolverProperties:
type: object
renderer:
type: string
rendererProperties:
Expand Down Expand Up @@ -5878,14 +5875,6 @@ definitions:
description: |-
Show property with templating resolved for the provided args.
If it resolves to false, the other fields are not set.
schema:
$ref: '#/definitions/v1StructType'
title: Schema of the resolved component data
data:
type: array
items:
type: object
title: Resolved component data
rendererProperties:
type: object
title: Renderer properties with templating resolved for the provided args
Expand Down
8 changes: 2 additions & 6 deletions proto/rill/runtime/v1/queries.proto
Original file line number Diff line number Diff line change
Expand Up @@ -779,20 +779,16 @@ message ResolveComponentRequest {
string instance_id = 1;
// Component name
string component = 2;
// Optional args to pass to the data resolver and for resolving templating in the renderer properties
// Optional args for resolving templating in the renderer properties
google.protobuf.Struct args = 3;
}

message ResolveComponentResponse {
// Show property with templating resolved for the provided args.
// If it resolves to false, the other fields are not set.
bool show = 1;
// Schema of the resolved component data
StructType schema = 2;
// Resolved component data
repeated google.protobuf.Struct data = 3;
// Renderer properties with templating resolved for the provided args
google.protobuf.Struct renderer_properties = 4;
google.protobuf.Struct renderer_properties = 2;
}

// **********
Expand Down
7 changes: 3 additions & 4 deletions proto/rill/runtime/v1/resources.proto
Original file line number Diff line number Diff line change
Expand Up @@ -671,8 +671,6 @@ message Component {
message ComponentSpec {
string display_name = 1;
string description = 7;
string resolver = 2;
google.protobuf.Struct resolver_properties = 3;
string renderer = 4;
google.protobuf.Struct renderer_properties = 5;
repeated ComponentVariable input = 8;
Expand All @@ -698,8 +696,9 @@ message Canvas {

message CanvasSpec {
string display_name = 1;
uint32 columns = 2;
uint32 gap = 3;
uint32 max_width = 2;
string theme = 7;
ThemeSpec embedded_theme = 8;
repeated ComponentVariable variables = 5;
repeated CanvasItem items = 4;
repeated SecurityRule security_rules = 6;
Expand Down
2 changes: 0 additions & 2 deletions runtime/compilers/rillv1/connectors.go
Original file line number Diff line number Diff line change
Expand Up @@ -81,8 +81,6 @@ func (a *connectorAnalyzer) analyzeResource(ctx context.Context, r *Resource) {
a.trackConnector(r.MigrationSpec.Connector, r, false)
} else if r.APISpec != nil {
a.analyzeResourceWithResolver(r, r.APISpec.Resolver, r.APISpec.ResolverProperties)
} else if r.ComponentSpec != nil {
a.analyzeResourceWithResolver(r, r.ComponentSpec.Resolver, r.ComponentSpec.ResolverProperties)
} else if r.AlertSpec != nil {
a.analyzeResourceNotifiers(r, r.AlertSpec.Notifiers)
} else if r.ReportSpec != nil {
Expand Down
Loading
Loading