Skip to content

Fix resource references not correctly resolved in apps config section#4964

Merged
andrewnester merged 8 commits intomainfrom
fix/app-schema-ref
Apr 14, 2026
Merged

Fix resource references not correctly resolved in apps config section#4964
andrewnester merged 8 commits intomainfrom
fix/app-schema-ref

Conversation

@andrewnester
Copy link
Copy Markdown
Contributor

Changes

Fix resource references not correctly resolved in apps config section

Why

Fixes #4962

Tests

Added an acceptance test

@andrewnester andrewnester requested review from denik and pietern April 14, 2026 10:34
Copy link
Copy Markdown
Contributor

@denik denik left a comment

Choose a reason for hiding this comment

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

FYI, this won't work for $resources references pointing to remote field (not present in the config).

@andrewnester andrewnester enabled auto-merge April 14, 2026 12:52
@andrewnester andrewnester added this pull request to the merge queue Apr 14, 2026
Merged via the queue into main with commit 876287c Apr 14, 2026
22 checks passed
@andrewnester andrewnester deleted the fix/app-schema-ref branch April 14, 2026 15:36
deco-sdk-tagging bot added a commit that referenced this pull request Apr 15, 2026
## Release v0.297.0

### CLI
* Auth commands now accept a profile name as a positional argument ([#4840](#4840))

* Add `auth logout` command for clearing cached OAuth tokens and optionally removing profiles ([#4613](#4613), [#4616](#4616), [#4647](#4647))

### Bundles
* Added support for lifecycle.started option for apps ([#4672](#4672))
* engine/direct: Fix permissions for resources.models ([#4941](#4941))
* Fix resource references not correctly resolved in apps config section ([#4964](#4964))
* Allow run_as for dashboards with embed_credentials set to false ([#4961](#4961))
* direct: Pass changed fields into update mask for apps instead of wildcard ([#4963](#4963))
* engine/direct: Fix deploy of configurations with dots in maps keys ([#4977](#4977))
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.

Resources are not replaced in env variables

2 participants