Skip to content

add my to applicable_formats#1

Open
ak4t0sh wants to merge 2 commits intovadimonus:masterfrom
ak4t0sh:master
Open

add my to applicable_formats#1
ak4t0sh wants to merge 2 commits intovadimonus:masterfrom
ak4t0sh:master

Conversation

@ak4t0sh
Copy link
Copy Markdown

@ak4t0sh ak4t0sh commented Jun 8, 2016

add possibility to add this block on my/dashboard page.

@vadimonus
Copy link
Copy Markdown
Owner

Excuse me for not responding for so long. I don't clearly understand, for what is needed stats block on dashboard page. On first page block shows overall site visits. On course page it shows course visits. What should it show on dashboard page?

@ak4t0sh
Copy link
Copy Markdown
Author

ak4t0sh commented Aug 15, 2016

What should it show on dashboard page?

I deployed this patch on some of my moodle instances it logically show same stats as on frontpage because $COURSE->id = 1 on the dashboard. I think this behavior is fine.
Indeed on most of moodle instances i use the dashboard/my as default page and as administrator i like to have quick access to several data such as usage statistic ASAP so i add several blocks (like graph_stats) on it for this.

Note : I think this PR could (and maybe should) be completed by an upgrade into db/access.php to prevent all users to add this block on their dashboard by default.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants