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

Show aggregated rights table and graph on sample page #49

Closed
5 tasks done
BBGuy opened this issue Oct 31, 2014 · 4 comments
Closed
5 tasks done

Show aggregated rights table and graph on sample page #49

BBGuy opened this issue Oct 31, 2014 · 4 comments
Assignees

Comments

@BBGuy
Copy link
Member

BBGuy commented Oct 31, 2014

Develop

  • create sample_asset context
  • Update code to handle sample page
    Deploy
  • Pull code
  • Import sample_asset context

Improvements

  • wen changing the values in the slider the split values are lost and when changing the split the slider values are lost - Sufficiently improved see note below
@BBGuy BBGuy assigned BBGuy and dahacouk and unassigned BBGuy and dahacouk Oct 31, 2014
@BBGuy
Copy link
Member Author

BBGuy commented Oct 31, 2014

We have an Issue where the url query parameters are not completable.
On asset page used for split percentage
/content/test?Composition=45&Performance=50&Recording=5

On sample page used by the slider
ample_asset/186?range[value]=0&range[value2]=91
will possibly need to have add code to handle the combination of the two for the sample page

@BBGuy
Copy link
Member Author

BBGuy commented Oct 31, 2014

I have implemented this on live now only one issue and that's the views slider (exposed form values) and the drop-down (inside the custom block form) don't work well together - if one is used the other one is reset.
should be easy to fix in the custom code but views may be more difficult.

@BBGuy
Copy link
Member Author

BBGuy commented Oct 31, 2014

Also fixed "Undefined index: contributions" errors on an empty asset see http://hub2.kendra.org/content/empty

@BBGuy
Copy link
Member Author

BBGuy commented Oct 31, 2014

I fixed the issue above for the split preserving the range looks like that should so the job as it is the second value - set your start and end then play with the drop-down
P.S all commits are against #48

@BBGuy BBGuy closed this as completed Dec 16, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants