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

Userlog Service #5610

Merged
merged 3 commits into from
Feb 23, 2023
Merged

Userlog Service #5610

merged 3 commits into from
Feb 23, 2023

Conversation

kobergj
Copy link
Collaborator

@kobergj kobergj commented Feb 21, 2023

Backport number 2 from #5506

Introduces the userlog service

mmattel
mmattel reviewed Feb 21, 2023
View reviewed changes
services/userlog/README.md Outdated Show resolved Hide resolved
@kobergj
introduce userlog service
d565655 
Signed-off-by: jkoberg <[email protected]>
@kobergj
changelog
d7f57f3 
Signed-off-by: jkoberg <[email protected]>
@kobergj kobergj force-pushed the UserlogService branch 2 times, most recently from 52fc0db to 6346800 Compare February 21, 2023 15:12
mmattel
mmattel reviewed Feb 21, 2023
View reviewed changes
services/userlog/README.md Outdated Show resolved Hide resolved
@ownclouders
Copy link
Contributor

ownclouders commented Feb 21, 2023 •
edited
Loading

💥 Acceptance test Core-API-Tests-ocis-storage-1 failed. Further test are cancelled...

@kobergj kobergj force-pushed the UserlogService branch 4 times, most recently from be8a89f to 0ecbfae Compare February 22, 2023 15:17
mmattel
mmattel reviewed Feb 23, 2023
View reviewed changes
services/userlog/README.md Outdated Show resolved Hide resolved
mmattel
mmattel reviewed Feb 23, 2023
View reviewed changes
services/userlog/README.md Outdated Show resolved Hide resolved
@mmattel mmattel mentioned this pull request Feb 23, 2023
Merged
@kobergj kobergj marked this pull request as ready for review February 23, 2023 13:24
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 9 Code Smells

30.0% 30.0% Coverage
1.0% 1.0% Duplication

@kobergj kobergj requested review from rhafer and removed request for phil-davis and individual-it February 23, 2023 13:42
@kobergj kobergj requested review from butonic and C0rby February 23, 2023 13:42
@mmattel
Copy link
Contributor

mmattel commented Feb 23, 2023

The texts and descriptions are ok from my side.

@mmattel mmattel merged commit fd5790d into owncloud:master Feb 23, 2023
ownclouders pushed a commit that referenced this pull request Feb 23, 2023
@mmattel
commit fd5790d
f81a558 
Merge: 3a07e14 a9561d8
Author: Martin <[email protected]>
Date:   Thu Feb 23 15:33:12 2023 +0100

    Merge pull request #5610 from kobergj/UserlogService

    Userlog Service
@micbar micbar mentioned this pull request May 3, 2023
Closed
89 tasks
@ScharfViktor ScharfViktor mentioned this pull request May 4, 2023
Closed
86 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mmattel mmattel mmattel left review comments

@C0rby C0rby C0rby approved these changes

@wkloucek wkloucek Awaiting requested review from wkloucek

@micbar micbar Awaiting requested review from micbar

@rhafer rhafer Awaiting requested review from rhafer

@butonic butonic Awaiting requested review from butonic

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@kobergj @ownclouders @mmattel @C0rby