Skip to content

Commit 35ca1d0

Browse files
committed
fix: Expose isVisible and public properties in meta consistently
1 parent 3e1a075 commit 35ca1d0

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

packages/cubejs-api-gateway/src/gateway.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -498,7 +498,6 @@ class ApiGateway {
498498
return metaConfig
499499
.map((cube) => ({
500500
config: {
501-
public: cube.isVisible,
502501
...cube.config,
503502
measures: cube.config.measures?.filter(visibilityFilter),
504503
dimensions: cube.config.dimensions?.filter(visibilityFilter),

packages/cubejs-schema-compiler/src/compiler/CubeToMetaTransformer.js

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,17 +36,19 @@ export class CubeToMetaTransformer {
3636
const isCubeVisible = this.isVisible(cube, true);
3737

3838
return {
39-
isVisible: isCubeVisible,
4039
config: {
4140
name: cube.name,
4241
type: cube.isView ? 'view' : 'cube',
4342
title: cubeTitle,
43+
isVisible: isCubeVisible,
44+
public: isCubeVisible,
4445
description: cube.description,
4546
connectedComponent: this.joinGraph.connectedComponents()[cube.name],
4647
measures: R.compose(
4748
R.map((nameToMetric) => ({
4849
...this.measureConfig(cube.name, cubeTitle, nameToMetric),
49-
isVisible: isCubeVisible ? this.isVisible(nameToMetric[1], true) : false
50+
isVisible: isCubeVisible ? this.isVisible(nameToMetric[1], true) : false,
51+
public: isCubeVisible ? this.isVisible(nameToMetric[1], true) : false
5052
})),
5153
R.toPairs
5254
)(cube.measures || {}),
@@ -61,7 +63,8 @@ export class CubeToMetaTransformer {
6163
nameToDimension[1].suggestFilterValues == null ? true : nameToDimension[1].suggestFilterValues,
6264
format: nameToDimension[1].format,
6365
meta: nameToDimension[1].meta,
64-
isVisible: isCubeVisible ? this.isVisible(nameToDimension[1], !nameToDimension[1].primaryKey) : false
66+
isVisible: isCubeVisible ? this.isVisible(nameToDimension[1], !nameToDimension[1].primaryKey) : false,
67+
public: isCubeVisible ? this.isVisible(nameToDimension[1], !nameToDimension[1].primaryKey) : false,
6568
})),
6669
R.toPairs
6770
)(cube.dimensions || {}),
@@ -72,7 +75,8 @@ export class CubeToMetaTransformer {
7275
shortTitle: this.title(cubeTitle, nameToSegment, true),
7376
description: nameToSegment[1].description,
7477
meta: nameToSegment[1].meta,
75-
isVisible: isCubeVisible ? this.isVisible(nameToSegment[1], true) : false
78+
isVisible: isCubeVisible ? this.isVisible(nameToSegment[1], true) : false,
79+
public: isCubeVisible ? this.isVisible(nameToSegment[1], true) : false,
7680
})),
7781
R.toPairs
7882
)(cube.segments || {})

0 commit comments

Comments
 (0)