Commit Graph

7 Commits

Author SHA1 Message Date
Ethan Vizitei c8b1c36d6f refs #8395 expanded page views
force a page_view record when
submitting an assignment
and when updating an assignment
description

force a page_view when
starting a new discussion
or when replying to a discussion

force a page_view when updating
a wiki page and when joining
a conference and when starting
a collaboration

start tracking http methods for
page views

don't link through to urls
for page views if they aren't 'get'
methods

pull some of the page_view logging
down into the asset_user_access
object and simplify it so
that the application controller is
more readable and the scoring incrementing
is easier to understand

TEST PLAN:
  1) login as a student
  2) turn in an assignment submission
  3) take a quiz
  4) update an assignment description
  5) start a discussion and reply to it
  6) login as an admin and confirm through
     analytics and through the detailed
     page view listing that all these
     actions counted as participations
     (and that the ones that are not
      'get' methods can't be clicked through
      from the student accound details page)

Change-Id: I2dfe4df41e8b006e6a0c0d8a1f914ecd7fb62477
Reviewed-on: https://gerrit.instructure.com/15428
Reviewed-by: Jacob Fugal <jacob@instructure.com>
Tested-by: Jenkins <jenkins@instructure.com>
2012-11-30 08:34:18 -07:00
Jacob Fugal 8115668ce6 let (AssetUserAccess|PageView).for_users take ids
and take AssetUserAccess.for_users back out since it's redundant

test-plan:
 - all works as before
 - you can now do PageView.for_users([a_user_id]) without error
 - and AssetUserAccess.for_user(a_user_id)

Change-Id: Ic9abc5bc3a0d376ab4386a780e1fc74446efeab4
Reviewed-on: https://gerrit.instructure.com/15295
Reviewed-by: Brian Palmer <brianp@instructure.com>
Tested-by: Jenkins <jenkins@instructure.com>
2012-11-14 11:34:29 -07:00
Jacob Fugal 85a72daf93 add some named scopes for convenience
also, tweak Assignment#overdue? and expand its use

Change-Id: I664765584047fa630c7048a8a702b8b2d1307361
Reviewed-on: https://gerrit.instructure.com/15153
Tested-by: Jenkins <jenkins@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
2012-11-09 09:47:14 -07:00
Brian Palmer 0b570f89f9 remove dead code around page views
Removes unused methods and associations from models, and removes the
PageViewRange model completely, which is no longer used.

test plan:
  * run the specs
  * verify that no active page-view-related functionality was inadvertently removed

Change-Id: I8522d151555db64a0ad2548334ae642f9fefd54e
Reviewed-on: https://gerrit.instructure.com/9324
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Cody Cutrer <cody@instructure.com>
2012-03-15 08:46:51 -06:00
Zach Wily 834b8517f0 fix names on the access report, other tweaks; fixes #4682
This makes the access report actually show the right names for things, and it
also displays icons next to them to indicate the type.

Change-Id: I0241bc9546555a0796dee35786cdfcddb930ef1a
Reviewed-on: https://gerrit.instructure.com/3869
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: JT Olds <jt@instructure.com>
2011-11-01 11:56:54 -06:00
Cody Cutrer bdbebfaec8 check all models for protecting attributes refs #3847
Change-Id: I7cba6e26ad98e91723e2ccf0a28b8db79bb37b5c
Reviewed-on: https://gerrit.instructure.com/3631
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Jon Jensen <jon@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
2011-05-25 17:38:50 -06:00
Brian Whitmer 8b8173dcc9 Initial commit.
closes #6988138
2011-01-31 18:57:29 -07:00