Skip to content

Commit

Permalink
feat: remove site_capabilities from the deploy object
Browse files Browse the repository at this point in the history
This was used just for large media and we're deprecating it. We'll be
removing this slowly and this is a good first step and allows us to
return a smaller object for deploys.

Updated the porcelain handler to not rely on this field but only on the
option passed on DoDeploy. This way we allow for builds to still use it
if needed until fully retired.
  • Loading branch information
kitop committed Jan 19, 2024
1 parent 54de2e6 commit c301715
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 34 deletions.
25 changes: 0 additions & 25 deletions go/models/deploy.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 0 additions & 4 deletions go/porcelain/deploy.go
Original file line number Diff line number Diff line change
Expand Up @@ -209,10 +209,6 @@ func (n *Netlify) DoDeploy(ctx context.Context, options *DeployOptions, deploy *
}

largeMediaEnabled := options.LargeMediaEnabled
if !largeMediaEnabled && deploy != nil {
largeMediaEnabled = deploy.SiteCapabilities.LargeMediaEnabled
}

ignoreInstallDirs := options.Dir == options.BuildDir

context.GetLogger(ctx).Infof("Getting files info with large media flag: %v", largeMediaEnabled)
Expand Down
5 changes: 0 additions & 5 deletions swagger.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3056,11 +3056,6 @@ definitions:
type: boolean
review_url:
type: string
site_capabilities:
type: object
properties:
large_media_enabled:
type: boolean
framework:
type: string
function_schedules:
Expand Down

0 comments on commit c301715

Please sign in to comment.