Skip to content

Several improvements (watching slots, import via CSV, mobile support, ...) - #42

Open
FMCorz wants to merge 29 commits into
learnweb:masterfrom
branchup:rca
Open

Several improvements (watching slots, import via CSV, mobile support, ...)#42
FMCorz wants to merge 29 commits into
learnweb:masterfrom
branchup:rca

Conversation

@FMCorz

@FMCorz FMCorz commented Mar 31, 2020

Copy link
Copy Markdown

Hi,

The Royal College of Art have sponsored the development of additional features in the Scheduler module and wished to share those back. Here is a rather large pull request which includes them all for your consideration.

We attempted to minimise the impact of these changes on the existing code in order to facilitate its integration in the main repository. We purposely did not refactor some areas of the code and opted for duplicated logic to avoid adding regressions, and added complexity.

Here is a list of the features which were included:

  • Additional Activity completion criterion based on attendance
  • Importing slots from a CSV file
  • Revoking individuals out of a group slot
  • Adding support for sorting the list of slots
  • Adding support for filtering the list of slots
  • Give the ability for students to watch slots that are full, and receive a notification when they become available. The number of slots that can be watched at the same time is customisable.
  • Mobile support for teachers to view slots, grade and mark attendance
  • Mobile support for students to view and book slots. (Except for bookings requiring file upload)

Thank you for your consideration.

This was developed and tested on Moodle 3.7 and is currently used in production with Moodle 3.8.

@FMCorz FMCorz changed the title Several improvements (watching slots, import via CSV, ...) Several improvements (watching slots, import via CSV, mobile support, ...) Mar 31, 2020
@jtsafran

Copy link
Copy Markdown

@bostelm Thoughts on integrating this into master? The use of this in mobile would be awesome!

@ptramonti

Copy link
Copy Markdown

We are also interested in the integration into the master! Can we know if it is your intentions?

@bostelm

bostelm commented Aug 13, 2021

Copy link
Copy Markdown
Collaborator

Hello all,
first, thanks a lot for sharing your work. Second, I'm sorry for not responding earlier - time pressures due to the pandemic meant that I wasn't able to prepare a new major release of Scheduler (which is where this would go). Third, yes, I'm interested in integrating this (possibly in parts). I hope I can look at it soon. Meanwhile, if you could rebase the patch to current master - it's been a while - that would be helpful.

@ptramonti

Copy link
Copy Markdown

Hello,
any news on this integration?

@lucaboesch

Copy link
Copy Markdown
Contributor

@FMCorz any chance those improvements could still be shared back, even if it has been some time ago?

Best,
Luca

@FMCorz

FMCorz commented Nov 28, 2022

Copy link
Copy Markdown
Author

Hi @lucaboesch,

The improvements have been shared back a while ago. It is unfortnate that they haven't been integrated yet, but at the moment I am unable to invest the time required to properly rebase the branch and test all of the added features.

Kind regards,
Fred

@FMCorz

FMCorz commented Aug 14, 2023

Copy link
Copy Markdown
Author

I meant to leave a comment here a few months ago, but apparently I forgot to publish it. The branch had been rebased. And more changes are pending in a separate branch.

@FMCorz

FMCorz commented Sep 14, 2023

Copy link
Copy Markdown
Author

This branch (rca) and rca-more have together been rebased on master in rca-more-rebased.

@Buda9

Buda9 commented Jul 16, 2024

Copy link
Copy Markdown

Thank you so much for this @FMCorz. Love this PR and your products! 🎉

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.

6 participants