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

1777 Project Largo Help dashboard widget updates #1778

Merged
merged 10 commits into from
Mar 23, 2021

Conversation

benlk
Copy link
Collaborator

@benlk benlk commented Aug 16, 2019

Changes

This pull request makes the following changes:

Screen Shot 2019-08-16 at 17 35 27

Why

The email signup link becomes a button as a more-obvious CTA in order to acquire more leads there. This way people are encouraged to sign up for the Largo updates newsletter, giving us both points of contact at sites as well as site domain names. It's an opt-in method that replaces the Google Analytics bug we removed for #1495.

Translatability for the translation god! Localization for the localized throne!

Testing/Questions

Features that this PR affects:

  • the admin dashboard

Questions that need to be answered before merging:

  • Do we want to further revise the wording here?

Steps to test this PR:

  1. Go to /wp-admin/ and make sure that the dashboard widget is visible.

@benlk benlk added this to In progress in Largo 0.7 Aug 21, 2019
@benlk benlk added this to To do in 0.7 PR review board Dec 19, 2019
@joshdarby joshdarby moved this from To do to In progress in 0.7 PR review board Jan 3, 2020
Largo 0.7 automation moved this from In progress to Reviewer approved Jan 3, 2020
@joshdarby joshdarby self-assigned this Mar 20, 2021
@joshdarby
Copy link
Collaborator

9a1f2b0 takes out the INN_MEMBER conditionals for the dashboard, removes the INN news dashboard widget and also replaces all links from INN links to WP Buddy links, which kind of helps kick off #1880.

@joshdarby joshdarby changed the title 1777 fix largo mailchimp link 1777 Project Largo Help dashboard widget updates Mar 20, 2021
@joshdarby joshdarby moved this from In progress to To do in 0.7 PR review board Mar 20, 2021
@joshdarby joshdarby moved this from To do to In progress in 0.7 PR review board Mar 23, 2021
@joshdarby joshdarby merged commit fb65806 into trunk Mar 23, 2021
Largo 0.7 automation moved this from Reviewer approved to Done Mar 23, 2021
@joshdarby joshdarby moved this from In progress to Done in 0.7 PR review board Mar 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Largo 0.7
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants