Page MenuHomePhorge
Differential D4659

Init default DAV folders
ClosedPublic
Actions

Authored by machniak on Mar 4 2024, 11:09 AM.
Tags
None
Referenced Files
F15095325: D4659.id14079.diff
Sun, Sep 8, 12:18 AM
Unknown Object (File)
Fri, Sep 6, 2:14 PM
Unknown Object (File)
Fri, Sep 6, 12:05 AM
Unknown Object (File)
Thu, Sep 5, 3:21 AM
Unknown Object (File)
Sat, Aug 31, 12:44 PM
Unknown Object (File)
Sat, Aug 31, 8:16 AM
Unknown Object (File)
Fri, Aug 30, 4:16 PM
Unknown Object (File)
Thu, Aug 29, 12:04 AM
View All Files
Subscribers
mollekopf
Restricted Project

Details

Reviewers
mollekopf
Group Reviewers
Restricted Project
Commits
rK3ad714919340: Init default DAV folders
Summary

Separation of tasks and events is not possible via Cyrus configuration, so we do this from Cockpit

Diff Detail

Repository
rK kolab
Branch
dev/dav-folders-init
Lint
Lint Skipped
Unit
No Test Coverage
Build Status
Buildable 47898
Build 18174: arc lint + arc unit

Event Timeline

machniak requested review of this revision.Mar 4 2024, 11:09 AM
machniak created this revision.
Harbormaster completed remote builds in B46716: Diff 13380.Mar 4 2024, 11:09 AM
machniak updated this revision to Diff 13812.Jun 28 2024, 8:21 AM
  • Merge branch 'master' into dev/dav-folders-init
Harbormaster completed remote builds in B47898: Diff 13812.Jun 28 2024, 8:21 AM
machniak updated this revision to Diff 14007.Mon, Aug 19, 12:49 PM
  • Merge branch 'master' into dev/dav-folders-init
  • Proxy authorization for DAV
  • CS fixes
Harbormaster completed remote builds in B48639: Diff 14007.Mon, Aug 19, 12:49 PM
machniak updated this revision to Diff 14019.Mon, Aug 19, 5:21 PM
  • Make updating default DAV folders properties possible
Harbormaster completed remote builds in B48756: Diff 14019.Mon, Aug 19, 5:21 PM
machniak updated this revision to Diff 14022.Tue, Aug 20, 11:53 AM
  • Refactor
  • Merge branch 'master' into dev/dav-folders-init
Harbormaster completed remote builds in B48759: Diff 14022.Tue, Aug 20, 11:53 AM
machniak retitled this revision from [WIP] Init default DAV folders to Init default DAV folders.Tue, Aug 20, 2:18 PM
mollekopf accepted this revision.Wed, Aug 28, 2:13 PM
mollekopf subscribed.
mollekopf added inline comments.
src/app/Backends/DAV.php
337

10s feels like not enough time, I would do at least 60

src/app/Backends/Helper.php
1 ↗(On Diff #14022)

The point of having the function in the config is so that the defaults are configurable, so I'm not getting why you'd move that here instead.

This revision is now accepted and ready to land.Wed, Aug 28, 2:13 PM
Closed by commit rK3ad714919340: Init default DAV folders (authored by machniak). · Explain WhyThu, Aug 29, 2:25 PM
This revision was automatically updated to reflect the committed changes.
machniak added a commit: rK3ad714919340: Init default DAV folders.

Revision Contents
Changeset List

PathSize
src/
app/
Backends/
39 lines
Jobs/
User/
7 lines
config/
28 lines

Diff 13812

View Options

src/app/Backends/DAV.php

Loading...
View Options

src/app/Jobs/User/CreateJob.php

Loading...
View Options

src/config/services.php

Loading...