Skip to content

Conversation

MichaelArestad
Copy link
Contributor

Changes proposed in this Pull Request:

  • Changed description of ads group
  • Changed label text to remove "unit"

Testing instructions:

  • Go to Settings > Traffic > Ads

Before

image

After

image

CC @dbspringer for thoughts on the copy changes

@MichaelArestad MichaelArestad added [Feature] WordAds [Status] Needs Review This PR is ready for review. [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it labels Feb 14, 2017
@MichaelArestad MichaelArestad self-assigned this Feb 14, 2017
@dbspringer
Copy link
Member

Looks good to me! 🚢

@dbspringer dbspringer added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Feb 14, 2017
@jeherve jeherve added this to the Settings UI milestone Feb 14, 2017
@samhotchkiss samhotchkiss merged commit 14adcaa into feature/settings-overhaul Feb 15, 2017
@samhotchkiss samhotchkiss deleted the update/ads-copy branch February 15, 2017 04:43
@samhotchkiss samhotchkiss removed the [Status] Ready to Merge Go ahead, you can push that green button! label Feb 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] WordAds [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.

4 participants