Skip to content

Commit 1b3238c

Browse files
committed
feat(bolt-install): clarify why we get the manifest from the local file or app settings
1 parent 3bbec60 commit 1b3238c

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

internal/pkg/apps/install.go

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -369,8 +369,11 @@ func InstallLocalApp(ctx context.Context, clients *shared.ClientFactory, orgGran
369369
}
370370

371371
// When the BoltInstall experiment is enabled, we need to get the manifest from the local file
372-
// if the manifest source is local or if we are creating a new app. Otherwise, we get the manifest
373-
// from the app settings.
372+
// if the manifest source is local or if we are creating a new app. After an app is created,
373+
// app settings becomes the source of truth for remote manifests, so updates and installs always
374+
// get the latest manifest from app settings.
375+
// When the BoltInstall experiment is disabled, we get the manifest from the local file because
376+
// this is how the original implementation worked.
374377
var slackManifest types.SlackYaml
375378
if clients.Config.WithExperimentOn(experiment.BoltInstall) {
376379
manifestSource, err := clients.Config.ProjectConfig.GetManifestSource(ctx)

0 commit comments

Comments
 (0)