Skip to content

Commit

Permalink
Fix migration fail when excludeByName is not present in transformatio…
Browse files Browse the repository at this point in the history
…n.id == "organize" (perses#2332)

Signed-off-by: 7840vz <7840vz@users.noreply.github.com>
Signed-off-by: Akshay Iyyadurai Balasundaram <akshay.iyyadurai.balasundaram@sap.com>
  • Loading branch information
7840vz authored and ibakshay committed Oct 21, 2024
1 parent 77f6e57 commit 1132d16
Showing 1 changed file with 13 additions and 10 deletions.
23 changes: 13 additions & 10 deletions cue/schemas/panels/table/migrate.cue
Original file line number Diff line number Diff line change
Expand Up @@ -25,16 +25,19 @@ if #panel.type != _|_ if #panel.type == "table" {
// #var
// ][0]
// }
// if #panel.transformations != _|_ for transformation in #panel.transformations if transformation.id == "organize" {
// for excludedColumn, value in transformation.options.excludeByName if value {
// let name = {_nameBuilder & {#var: excludedColumn}}.output
// _settingsGatherer: "\(name)": hide: true
// }
// for technicalName, displayName in transformation.options.renameByName {
// let name = {_nameBuilder & {#var: technicalName}}.output
// _settingsGatherer: "\(name)": headers: "\(displayName)": true
// }
// }
//if #panel.transformations != _|_ for transformation in #panel.transformations if transformation.id == "organize" {
// if transformation.options.excludeByName != _|_ {
// for excludedColumn, value in transformation.options.excludeByName if value {
// let name = {_nameBuilder & {#var: excludedColumn}}.output
// _settingsGatherer: "\(name)": hide: true
// }}
// if transformation.options.renameByName != _|_ {
// for technicalName, displayName in transformation.options.renameByName {
// let name = {_nameBuilder & {#var: technicalName}}.output
// _settingsGatherer: "\(name)": headers: "\(displayName)": true
// }
// }
//}
// if #panel.fieldConfig.overrides != _|_ {
// for override in #panel.fieldConfig.overrides if override.matcher.id == "byName" && override.matcher.options != _|_ {
// for property in override.properties {
Expand Down

0 comments on commit 1132d16

Please sign in to comment.