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

some changes #15

Open
wants to merge 69 commits into
base: master
Choose a base branch
from
Open

some changes #15

wants to merge 69 commits into from

Conversation

cuttazy
Copy link

@cuttazy cuttazy commented Jun 6, 2018

  • get courses list only one time (unique query)

  • removed remote courses function (adapted for institutions with 1 moodle)

  • adapted overviews only for the favourites (less time of execution)

  • Merge changes "Add Privacy API support and some small css fixes"

cuttazy added 4 commits May 29, 2018 16:36
- removed remote courses function (adapted for institutions with 1 moodle)
- adapted overviews only for the favourites (less time of execution)
@thepurpleblob
Copy link
Owner

Thanks.

Just at a glance... there's quite a few commented out lines and it should be "your" not "you" in that text string.

I'll have a proper look when I have a bit more time!

@cuttazy
Copy link
Author

cuttazy commented Jun 6, 2018

Yes, I'm still working on. I want to show how to fix deprecated functions too. Thank's for your feedback

cuttazy added 5 commits June 7, 2018 08:35
- cursor to drag&drop on elements
- Background color for courses with updates with admin setting
- Translations IT, FR, DE
- bug reorder drag&drop in favourites
@cuttazy
Copy link
Author

cuttazy commented Jun 19, 2018

  • Overview in courses list ON/OFF with setting for admin
  • Background color for courses with updates with admin setting
  • Translations IT, FR, DE
  • bug reorder drag&drop in favourites

cuttazy added 3 commits June 28, 2018 12:21
- changed style of close button in popup
- Default tab setting
@thepurpleblob
Copy link
Owner

Sorry - I'm getting a bit lost with this. I've sorted a few of these things myself and we've moved on a version or two. Any chance of a rebase/update?

@thepurpleblob
Copy link
Owner

Getting loads of what looks like development stuff pushed to this. Not sure if that's what you intended.

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