Link uses api client config as fallback #3642
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WHY are these changes introduced?
Last days users are reporting that the cli is returning this error when running whatever
app
commandAn example here
The issue was introduced in the version
3.56.0
by this PRMost of the
app
commands runs thelink
logic under the hood to sync the local config with the remote one as a first step. The problem is that there are lot of apps that doesn't have any active app version so the final configuration written in thetoml
in these scenarios is invalid.Before the versioned app config project was rolled out, a backfill process was run for the most important/installed apps. This process created a new app version using the API Client configuration to populate the
configuration
app modules.However, there are bunch of apps that the process didn't include in that first stage. In the following weeks the process will be executed for the rest of apps.
In the meantime we need a fix for any app that doesn't have any app version or the existing ones don't include any
configuration
app moduleWHAT is this pull request doing?
link
logic is using the API Client configuration content as a fallback in case the app doesn't have any app versionHow to test your changes?
app
command for an old remote app that doesn't include anyconfiguration
app modules. They execution should succeedapp
command for an old remote app that includes someconfiguration
app modules. They execution should succeedPost-release steps
Measuring impact
How do we know this change was effective? Please choose one:
Checklist
dev
ordeploy
have been reflected in the internal flowchart.