1953 distance shared #1996

gbrady92 · 2014-06-16T19:56:18Z

Hey Anil and Gina,
I've completed the changes suggested for this insight. These include:

The insight is now Twitter only, after looking into the hashtags and the style of posts the apps made on other networks, only Twitter was a suitable network.
Because the insight only supports Twitter I removed the code that checked for duplicate posts on different networks.
I've changed the language of the copy to be more aimed at encouraging the user to stay active rather than beat records.
I've added thresholds which provide the user with different feedback depending on the distance that the user shared compared to the previous week.
The biggest change to this insight is I have removed the Google chart and now focus on contextual feedback with images related to the feedback being displayed. These are chosen at random so the user should get different feedback.
I have edited the InsightTerms class to be more accurate.

Thanks for all the feedback on this insight, hopefully you enjoy the changes as much as I have.

TODO: * Redesign OSP admin views for plugins other than Facebook and Twitter * Delete old dashboard and menu interfaces, classes, registrars, tests * Add missing test coverage * Clean up code

…ctly

…n emails

…note in the footer

- URL-encode network username and network on insight headline links - Show correct network icon for each insight - Don't replace the colon in insight headlines - Improve test coverage Closes ThinkUpLLC#1809

When a non-logged-in user accesses an AuthController, redirect to the login page with a redirect GET parameter. On login, redirect the user to that original URL.

Add Smarty plugin to custom URL encode network usernames with spaces and accented characters

…ombs Insight fix: Don't show negative F-bombs

Nudges a user to post a photo if they haven't in the past week, as long as they have in the past two weeks. References ThinkUpLLC#1619

anildash · 2014-06-17T11:14:36Z

webapp/plugins/insightsgenerator/insights/distanceshared.php

+            } elseif($mile_diff == 0) {
+                $threshold_string = "That's exactly the same as last week!";
+            } else {
+                $threshold_string = "That's still better then everyone who decided to stay on the couch.";


I like this one, but we want "than", not "then". :)

anildash · 2014-06-17T11:15:40Z

I really love the changes here -- seems like you nailed the spirit of the thing. Off to @ginatrapani for code review, because I think we're good on copy and images.

Simple Javascript tool to preview various headlines and bodies emebdded in the html of an insight.

After a week since the last time the insight was generated, if there are new memberships, any missed in the past week are shown. Also adds additional headline options.

Insight text tester

…hip-repitition Insight fix: Reduce group membership repetition

Insight fix: Fix singular like in weekly graph insight

…me-me-or-mi-mi-mi-do-re Insight fix: Handle .me TLD more comprehensively in all about me Closes ThinkUpLLC#1991

…ight Photo Prompt Insight closes ThinkUpLLC#1619

…ll-noisy Stem Activity spike insight noise

Added config file, README docs for insight-html-to-server

…to-dislike-these-follower-count-tests Use mktime to calculate relative months for follower count tests.

They're sort of faux-insights in a new template, for users on thinkup.com. Copy fixes and code cleanup around the first run experience.

How often do you post super excitidely!!?!?!!? Lets you know if you used exclamation points in the past 30 days. Sometimes has a bar chart!!!

How often do you talk about the network on the network. Closes ThinkUpLLC#1977, closes ThinkUpLLC#2011

Once, not 1 times, Punctuation and timing.

gbrady92 · 2014-07-14T15:42:17Z

Suggested changes have been made to this insight. Hopefully it is is good to go @cdmoyer .

cdmoyer and others added 30 commits April 3, 2014 14:01

Added offset to times in instgram crawler test.

2259c06

ThinkUp LLC hosted app redesign

f53e381

TODO: * Redesign OSP admin views for plugins other than Facebook and Twitter * Delete old dashboard and menu interfaces, classes, registrars, tests * Add missing test coverage * Clean up code

Avoid mixed content warnings when running app on SSL

829f5a5

Email redesign assets [ci skip]

3414b49

Email redesign

da3dbd0

Email fixes - Date, header copy, unsub link

7b539d3

Reintroduce privacy setting

9a40295

Utils function to determine if ThinkUp LLC endpoint is set

5ebf1cf

Set from and unsub link in email notifications from thinkup.com corre…

95437ef

…ctly

Implement random subject lines and header text in insight notificatio…

d1b231c

…n emails

Fix randomizer range bug

b38dfe5

Set from: address to team@thinkup.com for ThinkUp LLC

cfb65fc

Session login API: don't include success message on successful redirect

c2c5279

Make email notification footer copy more informative

7989160

Flashback insight: Add missing period, clean code, add tests

d78c202

Add Settings to navigation menu

3d51679

Insight email notifications: New, temporary beta message and a reply …

5731a3a

…note in the footer

Insight notification email fixes

bf58e40

- URL-encode network username and network on insight headline links - Show correct network icon for each insight - Don't replace the colon in insight headlines - Improve test coverage Closes ThinkUpLLC#1809

Don't link Facebook users to Twitter

271a974

Facebook plugin: correctly set/display auth error when there is one

e224d1a

Facebook plugin: match re-auth email copy to UI

8fb9446

Implement login redirection

cb2c5bc

When a non-logged-in user accesses an AuthController, redirect to the login page with a redirect GET parameter. On login, redirect the user to that original URL.

Use site_root_path as default redirect on login

867e29c

Remove REQUEST_URI dependency from login redirection

1b27985

Remove DashboardPlugin, PostDetailPlugin, MenuItem

7533ef0

Update precompiled email template

030637f

Add Smarty plugin to custom URL encode network usernames with spaces and accented characters

Test fix

293bd10

Send all email from team@thinkup.com if app is hosted

a887d5b

Send Facebook reauth email to owner, not admin

1c487a1

Enable ability to output insight HTML to file during testing

b8dfff8

ginatrapani and others added 2 commits June 16, 2014 14:32

Merge pull request ThinkUpLLC#1972 from cdmoyer/fix-negative-fewer-fb…

39f281d

…ombs Insight fix: Don't show negative F-bombs

Photo Prompt Insight

d76a394

Nudges a user to post a photo if they haven't in the past week, as long as they have in the past two weeks. References ThinkUpLLC#1619

anildash reviewed Jun 17, 2014
View reviewed changes

cdmoyer and others added 23 commits June 17, 2014 15:16

HTML Insight Text Tester

4c6dc4d

Simple Javascript tool to preview various headlines and bodies emebdded in the html of an insight.

Added InsightDAO::getMostRecentInsight($slug, $instance_id)

259fdd3

Added GroupMembershipDAO::getNewMembershipsSince

fc39544

Change List Membership Insight to be limited to once per week.

998fe39

After a week since the last time the insight was generated, if there are new memberships, any missed in the past week are shown. Also adds additional headline options.

Fix pluralization in weekly graph likes text.

45792f3

Handle .me tld in all about me more comprehensively.

7816209

Merge pull request ThinkUpLLC#1998 from cdmoyer/insight-text-tester

0c8bcf3

Insight text tester

Merge pull request ThinkUpLLC#2003 from cdmoyer/reduce-group_membders…

002dc7e

…hip-repitition Insight fix: Reduce group membership repetition

Merge pull request ThinkUpLLC#2004 from cdmoyer/fix-singular-weekly

5fb9c10

Insight fix: Fix singular like in weekly graph insight

Merge pull request ThinkUpLLC#2005 from cdmoyer/1991-dotme-is-not-me-…

5bfc11e

…me-me-or-mi-mi-mi-do-re Insight fix: Handle .me TLD more comprehensively in all about me Closes ThinkUpLLC#1991

Merge pull request ThinkUpLLC#2006 from cdmoyer/1619-photo-prompt-ins…

0d9f94f

…ight Photo Prompt Insight closes ThinkUpLLC#1619

Photo prompt: Remove first-run logic since this is a weekly insight

3f8dcc5

Fetch time_generated in InsightMySQLDAO::getInsight

d64c30f

Don't delete and recreate existing activityspike insights.

97e9e2c

Merge pull request ThinkUpLLC#2023 from cdmoyer/activity-spike-is-sti…

277d193

…ll-noisy Stem Activity spike insight noise

Developer extras script to generate and scp rendered insight html

07d6f17

Added config file, README docs for insight-html-to-server

Use mktime to calculate relative months for follower count tests.

27c3b0b

Merge pull request ThinkUpLLC#1921 from cdmoyer/i-really-am-starting-…

1924763

…to-dislike-these-follower-count-tests Use mktime to calculate relative months for follower count tests.

Add clearer first-run welcome message

3a04053

They're sort of faux-insights in a new template, for users on thinkup.com. Copy fixes and code cleanup around the first run experience.

Exclamation Count Insight

4a89d74

How often do you post super excitidely!!?!?!!? Lets you know if you used exclamation points in the past 30 days. Sometimes has a bar chart!!!

New insight; Meta Post Count

b6b4079

How often do you talk about the network on the network. Closes ThinkUpLLC#1977, closes ThinkUpLLC#2011

Meta post count insight fixes

e1ba8a3

Once, not 1 times, Punctuation and timing.

1953 distance shared insight. Complete with all fixes.

6ac86df

ginatrapani force-pushed the thinkup.com branch from 16b374f to 461d9f8 Compare September 4, 2014 17:53

gbrady92 closed this Feb 1, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

1953 distance shared #1996

1953 distance shared #1996

gbrady92 commented Jun 16, 2014

anildash Jun 17, 2014

anildash commented Jun 17, 2014

gbrady92 commented Jul 14, 2014

1953 distance shared #1996

1953 distance shared #1996

Conversation

gbrady92 commented Jun 16, 2014

anildash Jun 17, 2014

Choose a reason for hiding this comment

anildash commented Jun 17, 2014

gbrady92 commented Jul 14, 2014