Permalink
Commits on Oct 28, 2011
  1. Refactor auth mechanism for public reply search

    mwilkie committed with ginatrapani Oct 25, 2011
    * Add preAuthControl() functionality and tests for auth controller
    * move public auth mechanism into GridController->preAuthControl()
    Closes #1066, closes #1067
  2. Temporarily remove links to user page

    ginatrapani committed Oct 28, 2011
    Closes #1070
  3. Various UI improvements

    waxpancake committed with ginatrapani Oct 28, 2011
    On post pages, don't hide post metadata, embed, or wordcloud
    Modified to better fit Twitter's standards for metadata display
    Clean up post metadata formatting to match Twitter, don't hide wordcloud/embed by default
    Display advanced options underneath "Show advanced options" toggle, instead of above
    Open 'create a new application' links in new windows
    Add 'copy to clipboard' for all Callback URL settings during installation
    In Facebook plugin setup, modify instructions to reflect Facebook's renamed button
    Modify plugin setup instructions to reflect Facebook's changes to app submission
    During plugin setup, give examples for the fields during the submission process
    Closes #1071
  4. Dashboard sidebar menu fixes

    ginatrapani committed Oct 28, 2011
    * Facebook Plugin: Moved Wall posts from Friends to Posts
    * G+ Plugin: Fixed MenuItem declarations
    * Standardized view listing slugs (posts-all, posts-mostreplies, etc)
Commits on Oct 27, 2011
  1. Collapse the Dashboard sidebar menu

    ginatrapani committed Oct 25, 2011
    * Only show high-level categories for Twitter, Facebook, and G+ plugins
    * Display related modules under each high-level menu item in single view, with More... links to full listing (like the Dashboard)
    * Added breadcrumb nav to listings 2 deep, maintain sidebar selected state when 2 deep
    Closes #998
Commits on Oct 25, 2011
  1. Don't show protected posts to owners w/out permission

    ginatrapani committed Oct 25, 2011
    Users only see posts by protected users they have access to on the source network
    Administrators see all posts, even protected ones collected on other users' behalf
    This permission check has been implemented on the individual post page and on the replies listing
    Still TODO: Update the GridController and ExportController to return PostIterators without private posts owner doesn't have access to
    Partially addresses #648
Commits on Oct 24, 2011
  1. Add ability for non-authenticated user to search public post replies

    mwilkie committed with ginatrapani Oct 24, 2011
    - updated auth controller to pass though reply searches
    - added getById call and tests for OwnerDAO
    - Updated GridController and tests to support non-authed reply search
    - updated post.index.tpl to allow non authed reply searches
    Closes #826, closes #1063
  2. Fix uncaught API budget exception in twitter crawler

    mwilkie committed with ginatrapani Oct 23, 2011
    - Catch APICallLimitExceededException for TwitterCrawler->fetchInstanceUserGroups()
    Closes #1061
  3. Get wall posts from Facebook

    cwarden committed with ginatrapani Oct 13, 2011
    Pull /<id>/feed instead of /<id>/posts so wall posts by other users are
    retrieved along with posts by the user.
    
    Wall posts set the in_reply_to_user_id field on the post.
    
    Data files for /feed API calls were copied from /posts files, with a
    wall post added to Gina's feed.
    
    The dashboard for Facebook is called Posts to you.  For Twitter, it's
    called Tweets to you, and is also messages
    sent to a user, but not in response to a previous message.
    Closes #1059
  4. Warn user of slow web migration

    Mark Wilkie committed with ginatrapani Oct 5, 2011
    - Added TableStatsDAO and tests to glean table row counts
    - Added check for high table row counts to upgrade controller
    - Warn user of high row counts, and suggest CLI upgrade instead
    - Added check for high table row counts to backup controller
    - Warn user of high row counts, and suggest CLI backup instead
    Closes #688, closes #1058
  5. Group count quick fixes

    ginatrapani committed Oct 24, 2011
Commits on Oct 21, 2011
  1. [DB MIGRATION REQ'D] List support

    cwarden committed with ginatrapani Oct 4, 2011
    * Add DAO and MySQL DAO implementations for Group, Group Member, and Group Membership Count
    * Add Group class
    * Update Twitter crawler to record groups, group memberships, and snapshot total active group membership count for each instance
    * Add group membership count history tab to dashboard
    * Stale group memberships get marked as inactive
    * Not implemented: group ownership tracking, marking entire groups inactive
    
    Co-authored by @ginatrapani
    Closes #17, closes #1029
Commits on Oct 20, 2011
  1. Add Facebook pages you manage to dropdown list

    cwarden committed with ginatrapani Oct 11, 2011
    On the Facebook admin page, show pages for which you are the
    administrator in addition to the ones you "like".
    
    Both types of pages are shown in the same pulldown, separated into
    optgroups.
    
    Closes #1051
  2. Documentation: Troubleshooting crawls with the developer log

    joweiser committed with ginatrapani Oct 18, 2011
    Text taken from Gina's response in the Google Group: https://groups.google.com/d/msg/thinkupapp/x3kBcVraXJU/5wlzY-UQws8J
    Closes #1052
  3. Whitespace cleanup and conditional short-circuit

    cwarden committed with ginatrapani Oct 13, 2011
    No functional changes: just removes some extraneous whitespace and makes the
    conditional in storeFriends() smaller.
    Closes #1043
  4. Fix premature termination of conditional

    cwarden committed with ginatrapani Oct 20, 2011
    There was an extra {/if} tag after the followers-by-day chart img tag:
    https://github.com/cwarden/ThinkUp/blob/99c1d7e3776411b615386c9ed4cb8c123cade2ab/webapp/_lib/view/dashboard.tpl#L148
    
    Reformated to make it easier to follow (view this diff with whitespace
    changes suppressed to see logic changes) so we no longer have random
    placement of conditional statements:
    https://github.com/cwarden/ThinkUp/blob/99c1d7e3776411b615386c9ed4cb8c123cade2ab/webapp/_lib/view/dashboard.tpl#L208-211
    Closes #1054
  5. Handle SERVER_NAME not being set

    cwarden committed with ginatrapani Oct 14, 2011
    When constructing the redirect url to get an access token, use the
    HTTP_HOST header value if SERVER_NAME is not defined.  HTTP_HOST should
    actually be the default since this is what is used to construct the
    authorization url in BaseFacebook::getLoginUrl, but SERVER_NAME is kept
    as the default to make this change minimally intrusive; it fixes the
    case when SERVER_NAME is not defined by the web server and shouldn't
    break any cases that are currently working nor any of the existing
    tests.
    
    The HTTP_HOST was also changed in
    TestOfFacebookPluginConfigurationController::setUp to match SERVER_NAME
    since this is the normal case.  Other cases should be handled by special
    tests.
    Closes #1049
  6. Fix ability to add a Facebook account under https

    cwarden committed with ginatrapani Oct 14, 2011
    When running under https, the redirect url generated when creating an
    authorization url correctly set the protocol to https if the ThinkUp web
    server is running under https, but always set the redirect url protocol
    to http when creating the url to request an access token.  This caused
    Facebook to reject the request for the access token if the app was
    configured with the https url.
    Closes #1048
  7. Remove jquery being loaded from googleapis.com

    cwarden committed with ginatrapani Oct 19, 2011
    A few were missed in #819
    Closes #1053
  8. Minor Facebook crawler bug fix

    cwarden committed with ginatrapani Oct 13, 2011
    Misplaced right paren caused parsing of empty stream results.
    Closes #1042
Commits on Oct 13, 2011
  1. Don't load default instance on dashboard if unnecessary

    cwarden committed with ginatrapani Oct 12, 2011
    Only load a default instance on the dashboard if one wasn't specified in
    the URL, or the one specified is unaccessible.
    
    Closes #1041
Commits on Oct 10, 2011
  1. Test output fix

    ginatrapani committed Oct 10, 2011
  2. Version 0.16 (beta 16)

    ginatrapani committed Oct 10, 2011
  3. Multi-statement migration rollbacks

    Mark Wilkie committed with ginatrapani Oct 7, 2011
Commits on Oct 9, 2011
  1. [DB MIGRATION REQ'D] Defensively rewrite links_post_key migration

    ginatrapani committed Oct 9, 2011
    * Avoid 'Duplicate entry' failure when adding the unique key to the tu_links table by replacing the ALTER statement with CREATE TABLE + INSERT IGNORE
    
    Developers who already successfully ran the 2011-10-07 version will get this error: "Column already exists: 1060 Duplicate column name 'post_key'"
    To resolve it, run the following SQL:
    
    INSERT INTO tu_completed_migrations (migration) VALUES ('2011-10-09_links_post_key-0');
    INSERT INTO tu_completed_migrations (migration) VALUES ('2011-10-09_links_post_key-1');
    INSERT INTO tu_completed_migrations (migration) VALUES ('2011-10-09_links_post_key-2');
    INSERT INTO tu_completed_migrations (migration) VALUES ('2011-10-09_links_post_key-3');
    INSERT INTO tu_completed_migrations (migration) VALUES ('2011-10-09_links_post_key-4');
    INSERT INTO tu_completed_migrations (migration) VALUES ('2011-10-09_links_post_key-5');
    INSERT INTO tu_completed_migrations (migration) VALUES ('2011-10-09_links_post_key-6');
    INSERT INTO tu_completed_migrations (migration) VALUES ('2011-10-09_links_post_key-7');
    INSERT INTO tu_completed_migrations (migration) VALUES ('2011-10-09_links_post_key-8');
    INSERT INTO tu_completed_migrations (migration) VALUES ('2011-10-09_links_post_key-9');
    INSERT INTO tu_completed_migrations (migration) VALUES ('2011-10-09_links_post_key-10');
    INSERT INTO tu_completed_migrations (migration) VALUES ('2011-10-09_links_post_key-11');
    INSERT INTO tu_completed_migrations (migration) VALUES ('2011-10-09_links_post_key-12');
  2. Fix crawl RSS URL, closes #1027

    ginatrapani committed Oct 9, 2011
Commits on Oct 8, 2011
  1. Test failure fixes

    ginatrapani committed Oct 8, 2011
  2. [DB MIGRATION REQ'D] Re-widen post_id for G+

    ginatrapani committed Oct 7, 2011
    * Squash and rename 2 existing Google+ post/user ID migrations
    * Adjust post_id resize to VARCHAR( 80 ) to accommodate 76 character-long G+ comment IDs
    * This rename will force developers only to re-run the migration, but it won't corrupt data