Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add namecheap component #3589

Merged
merged 2 commits into from Oct 13, 2017
Merged

Add namecheap component #3589

merged 2 commits into from Oct 13, 2017

Conversation

Munsio
Copy link
Contributor

@Munsio Munsio commented Oct 11, 2017

Description:

Add the component site for the namecheapDNS

Pull request in home-assistant (if applicable): home-assistant/core#9821

@fabaff fabaff added Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! new-integration This PR adds documentation for a new Home Assistant integration labels Oct 12, 2017
@balloob balloob merged commit 1aed495 into home-assistant:next Oct 13, 2017

```yaml
# Example configuration.yaml entry
namecheapdns:
Copy link
Member

Choose a reason for hiding this comment

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

Rename this to subdomain please. This looks like a hack at the moment.

Copy link
Member

Choose a reason for hiding this comment

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

The component is called namecheapdns?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is there something to change or is this comment resolved cause the PR is already merged!?

Copy link
Member

Choose a reason for hiding this comment

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

Sorry, wrong line. i was referring to host.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

At the namecheap doc site they reference to the parameter as host so i thought if you use the component in addition to the documentation from namecheap you will know what you have to fill in there.

The only thing i turned around was the "access_token" they reference it as "dynamic DNS Password"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! new-integration This PR adds documentation for a new Home Assistant integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants