Skip to content
This repository was archived by the owner on Dec 9, 2020. It is now read-only.

Add support for 'domain_aliases' #30

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gozer
Copy link

@gozer gozer commented Jun 28, 2019

Can now support a list of domain aliases:

resource "netlify_site" "site" {
  [...]
  domain_aliases = [
    "www1.example.com",
    "www2.example.com",
  ]
}

Fixes #29

Can now support a list of domain aliases:

```
resource "netlify_site" "site" {
  [...]
  domain_aliases = [
    "www1.example.com",
    "www2.example.com",
  ]
}
```
Fixes hashicorp#29
@ziegeer
Copy link

ziegeer commented Oct 7, 2019

HI we depend on the domain aliases Netlify feature and would love to be able to use Terraform for it. Is there some way I can help get this merged? I can verify this PR code works well.

@dcow
Copy link

dcow commented Feb 4, 2020

Any update?

@limed
Copy link

limed commented Feb 6, 2020

Bump, any update on this?

@@ -27,6 +27,11 @@ func resourceSite() *schema.Resource {
Type: schema.TypeString,
Optional: true,
},
"domain_aliases": {
Type: schema.TypeList,
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be a set, or does the order matter here?

Copy link

Choose a reason for hiding this comment

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

I don't think order really matters in this scenario

@paultyng paultyng self-assigned this Apr 17, 2020
@yacinehmito
Copy link

Hello @paultyng. How can I help get this merged?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for domain aliases
6 participants