-
Notifications
You must be signed in to change notification settings - Fork 17
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
ISSUE-1048 [PublicAsset] PublicAsset/set create JMAP method #1057
Conversation
...ap/extensions/src/main/scala/com/linagora/tmail/james/jmap/method/PublicAssetSetMethod.scala
Outdated
Show resolved
Hide resolved
...ap/extensions/src/main/scala/com/linagora/tmail/james/jmap/method/PublicAssetSetMethod.scala
Outdated
Show resolved
Hide resolved
...ap/extensions/src/main/scala/com/linagora/tmail/james/jmap/method/PublicAssetSetMethod.scala
Outdated
Show resolved
Hide resolved
...ap/extensions/src/main/scala/com/linagora/tmail/james/jmap/method/PublicAssetSetMethod.scala
Outdated
Show resolved
Hide resolved
...ap/extensions/src/main/scala/com/linagora/tmail/james/jmap/method/PublicAssetSetMethod.scala
Outdated
Show resolved
Hide resolved
...ap/extensions/src/main/scala/com/linagora/tmail/james/jmap/method/PublicAssetSetMethod.scala
Outdated
Show resolved
Hide resolved
...ensions/src/main/scala/com/linagora/tmail/james/jmap/publicAsset/PublicAssetSetRequest.scala
Outdated
Show resolved
Hide resolved
...ensions/src/main/scala/com/linagora/tmail/james/jmap/publicAsset/PublicAssetSetRequest.scala
Show resolved
Hide resolved
...ensions/src/main/scala/com/linagora/tmail/james/jmap/publicAsset/PublicAssetSetService.scala
Outdated
Show resolved
Hide resolved
...ensions/src/main/scala/com/linagora/tmail/james/jmap/publicAsset/PublicAssetSetService.scala
Outdated
Show resolved
Hide resolved
...ensions/src/main/scala/com/linagora/tmail/james/jmap/publicAsset/PublicAssetSetService.scala
Outdated
Show resolved
Hide resolved
...ensions/src/main/scala/com/linagora/tmail/james/jmap/publicAsset/PublicAssetSetService.scala
Outdated
Show resolved
Hide resolved
...-tests-common/src/main/scala/com/linagora/tmail/james/common/PublicAssetMethodContract.scala
Outdated
Show resolved
Hide resolved
...-tests-common/src/main/scala/com/linagora/tmail/james/common/PublicAssetMethodContract.scala
Outdated
Show resolved
Hide resolved
...ap/extensions/src/main/scala/com/linagora/tmail/james/jmap/method/PublicAssetSetMethod.scala
Outdated
Show resolved
Hide resolved
...ensions/src/main/scala/com/linagora/tmail/james/jmap/publicAsset/PublicAssetSetService.scala
Outdated
Show resolved
Hide resolved
It will be reusing the james-project pr JAMES-3544 Add getUploadMetadata method for UploadRepository + add to UploadResolver |
b9cdf2e
to
933d890
Compare
[ERROR] /home/jenkins/build/workspace/Tmail_build_PR-1057/tmail-backend/jmap/extensions/src/main/scala/com/linagora/tmail/james/jmap/method/PublicAssetSetMethod.scala:134: value getUploadMetadata is not a member of org.apache.james.jmap.routes.UploadResolver |
Please don't! This is uneeded. As expressed there please rely solely on BlobResolvers @vttranlina |
But we want to fetch only metadata, while |
Some code is using code of james-project that was not merged. (commit: f165c07) |
Excuse me we want to copy the content as far as I know.... Because uploads will be cleaned up. Which means underlying blobId will disappear if nothing is done. |
933d890
to
a7d9af9
Compare
A lot of changed code, then I will rebase & force push. It will easier |
...-tests-common/src/main/scala/com/linagora/tmail/james/common/PublicAssetMethodContract.scala
Outdated
Show resolved
Hide resolved
...-tests-common/src/main/scala/com/linagora/tmail/james/common/PublicAssetMethodContract.scala
Outdated
Show resolved
Hide resolved
...ap/extensions/src/main/scala/com/linagora/tmail/james/jmap/method/PublicAssetSetMethod.scala
Show resolved
Hide resolved
...ap/extensions/src/main/scala/com/linagora/tmail/james/jmap/method/PublicAssetSetMethod.scala
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update the spec please ;-)
...map/extensions/src/main/scala/com/linagora/tmail/james/jmap/json/PublicAssetSerializer.scala
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM overall
...-tests-common/src/main/scala/com/linagora/tmail/james/common/PublicAssetMethodContract.scala
Outdated
Show resolved
Hide resolved
...-tests-common/src/main/scala/com/linagora/tmail/james/common/PublicAssetMethodContract.scala
Outdated
Show resolved
Hide resolved
...integration-tests/src/test/java/com/linagora/tmail/james/MemoryPublicAssetSetMethodTest.java
Outdated
Show resolved
Hide resolved
...ensions/src/main/scala/com/linagora/tmail/james/jmap/publicAsset/PublicAssetSetService.scala
Outdated
Show resolved
Hide resolved
docs/modules/ROOT/pages/tmail-backend/jmap-extensions/publicAssets.adoc
Outdated
Show resolved
Hide resolved
docs/modules/ROOT/pages/tmail-backend/jmap-extensions/publicAssets.adoc
Outdated
Show resolved
Hide resolved
docs/modules/ROOT/pages/tmail-backend/jmap-extensions/publicAssets.adoc
Outdated
Show resolved
Hide resolved
Initially the spec was good regarding /set create, so please revert that :) |
The Cassandra repository was merged |
…add Distributed Test
bcc6544
to
7a4d528
Compare
resolve #1048