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

Domains: Allow empty names for TXT and MX records #3235

Merged
merged 1 commit into from
Feb 15, 2016

Conversation

klimeryk
Copy link
Contributor

This partially fixes #234.
A separate PR is needed to add editing protected records (root A/AAAA or MX when email forwarding is enabled).

cc @umurkontaci @aidvu @bikedorkjon

@klimeryk klimeryk added [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. [Feature Group] Emails & Domains Features related to email integrations and domain management. labels Feb 11, 2016
@klimeryk klimeryk self-assigned this Feb 11, 2016
@klimeryk klimeryk force-pushed the fix/allow-blank-mx-txt-records branch from 9e7623e to 5afdc55 Compare February 11, 2016 02:51
@klimeryk klimeryk changed the title Domains: allow empty names for TXT and MX records Domains: Allow empty names for TXT and MX records Feb 11, 2016
@umurkontaci
Copy link
Contributor

Root TXT records work but root MX records are not working with a dig

@umurkontaci umurkontaci added [Status] Needs Author Reply and removed [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. labels Feb 15, 2016
@umurkontaci
Copy link
Contributor

^ Disregard. DNS propagation weirdness was the reason.

@umurkontaci
Copy link
Contributor

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature Group] Emails & Domains Features related to email integrations and domain management. [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Domains: DNS: Unable to add a root A record
3 participants