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.

Sign up for GitHub

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

Jump to bottom

orgu: #28535 add an activatable permission check for collection of or… #2769

Merged
merged 2 commits into from
Feb 21, 2022
Merged

orgu: #28535 add an activatable permission check for collection of or… #2769

merged 2 commits into from
Feb 21, 2022

Conversation

shecken
Copy link
Contributor

@shecken shecken commented Jun 25, 2020

…g units

@klees klees added the bugfix label Jun 25, 2020
@mjansenDatabay
Copy link
Contributor

@mstuder We stumbled upon this old PR in our TB meeting. Do you have any objections to merge this?

@chfsx
Copy link
Member

chfsx commented Apr 7, 2021

Hi @mstuder

this came up again in our last TB meeting on the 06-04-2021. Is there anything new regarding this PR? Is there anything we could do to speed up the process?

Best regards!

@chfsx
Copy link
Member

chfsx commented Jun 15, 2021

Hi @mstuder

this came up again in our last TB meeting on the 06-04-2021. Is there anything new regarding this PR? Is there anything we could do to speed up the process?

Best regards!

@Amstutz
Copy link
Contributor

Amstutz commented Aug 10, 2021

Hi @mstuder

this came up again in our last TB meeting on the 10-08-2021. Is there anything new regarding this PR? Is there anything we could do to speed up the process? As far as we understand your current position, you will take care of your components, until another maintainer will be found.

Best regards!

@chfsx
Copy link
Member

chfsx commented Nov 30, 2021

Hi @mstuder
As Technical Board, we regularly check for pull requests that have been open for a long time.
As the assignee, if you don't have any objections, please merge this PR or provide feedback otherwise.

Best regards!

@mstuder
Copy link
Contributor

mstuder commented Feb 21, 2022

thank you - I will merge this in the other branches to, this week!

@mstuder mstuder merged commit 378c31f into ILIAS-eLearning:release_6 Feb 21, 2022
mstuder pushed a commit to mstuder/ILIAS that referenced this pull request Mar 10, 2022
@shecken @mstuder
cherry-pick ILIAS-eLearning#2769
d36f12b
mstuder added a commit that referenced this pull request Mar 10, 2022
@mstuder @shecken
@mjansenDatabay @jeph864 @CrawdadWarden @M-Barz
Fixes/from ilias 6 (#4115)
9dab9a8 
* cherry-pick #2742

* Org.Units: Fix filter when listing courses for user (#4035)

* cherry pick #3768

* show enrolments for a single user (#3595)

* Allows custom positions to be imported. (#3113)

* Update class.ilOrgUnitSimpleUserImport.php

* cherry-pick #2778

* cherry-pick #2769

* cherry-pick #4078


Co-authored-by: Stefan Hecken <[email protected]>
Co-authored-by: Michael Jansen <[email protected]>
Co-authored-by: Jephte Abijuru <[email protected]>
Co-authored-by: Evan Jackson <[email protected]>
Co-authored-by: Marvin Barz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mstuder mstuder

Labels
bugfix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@shecken @mjansenDatabay @chfsx @Amstutz @mstuder @klees