Project

General

Profile

Bug #7382

Variable number of icons in reporting tab, admin role

Added by Yannick Warnier over 6 years ago. Updated almost 6 years ago.

Status:
Assigned
Priority:
Normal
Assignee:
Yannick Warnier
Category:
Exercises
Target version:
2.0
Start date:
08/11/2014
Due date:
% Done:

40%

Estimated time:
Spent time:
16.10 h
Complexity:
Normal
SCRUM pts - complexity:
?

Description

On the global reporting page, the admin sees a series of action icons, then he clicks one and the number or the order changes...

History

#1

Updated by Daniel Barreto over 6 years ago

  • Assignee changed from Yannick Warnier to Daniel Barreto
  • % Done changed from 0 to 60

Fix action bar for reports to keep order and the same actions
Add gray icons for stats icons

Send PR431
https://github.com/chamilo/chamilo-lms/pull/431

#2

Updated by Yannick Warnier over 6 years ago

From a very superficial review, it doesn't look like you're making any difference between the user roles. Maybe that would be an issue. Did you try it with each user role? Not all users have access to all the icons.

#3

Updated by Daniel Barreto over 6 years ago

Checking reports by roles

#4

Updated by Daniel Barreto over 6 years ago

Trying to integrate reports actions, need repair course reports too

Tomorrow should be completed

https://github.com/danbarretodev/chamilo-lms/tree/CT7382_1

#5

Updated by Daniel Barreto over 6 years ago

  • Assignee changed from Daniel Barreto to Yannick Warnier

Integrate report actions and options(print, export) in Myspace class with public static methods.
Fix breadcrumb from some reports
Fix a form from exam tracking course report
Add documentation for new methods

Update PR431

Please check it at
https://github.com/chamilo/chamilo-lms/pull/431

#6

Updated by Yannick Warnier over 6 years ago

  • Assignee changed from Yannick Warnier to Julio Montoya
#7

Updated by Julio Montoya over 6 years ago

  • Status changed from Assigned to Needs more info
  • Assignee changed from Julio Montoya to Yannick Warnier

This is a big change, I think is better to move this task to 1.9.12 in order to have the time to everyone to test it?

#8

Updated by Yannick Warnier over 6 years ago

  • Assignee deleted (Yannick Warnier)
  • Target version changed from 1.9.10 to 1.10.0

OK. I was hoping to include it in 1.9.10, but you're right, it is a big change.

#9

Updated by Yannick Warnier about 6 years ago

  • Status changed from Needs more info to Assigned
  • Assignee set to Julio Montoya
  • % Done changed from 60 to 40

I have generated a new PR from Daniel's CT7382_1 branch, to be analyzed

#10

Updated by Yannick Warnier almost 6 years ago

Julio, do you think we should try to finish and pull it, or that we're better starting again later on (considering many things might have changed in the code in the meantime)?

#11

Updated by Julio Montoya almost 6 years ago

  • Assignee changed from Julio Montoya to Yannick Warnier

A new PR has to be sent. I can't change the branch which was set to 1.9.x.

#12

Updated by Yannick Warnier almost 6 years ago

  • Target version changed from 1.10.0 to 2.0

Also available in: Atom PDF