-
Notifications
You must be signed in to change notification settings - Fork 0
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
Setup codebase for Notifications within the plugin. #71
base: main
Are you sure you want to change the base?
Conversation
Enables notifications for when files are uploaded (aka decentralized) to Swarm.
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.
I believe your changes were not in sync with the latest code, so some changes are reverting other PRs changes. Otherwise, the notification changes look good code wise to me.
@@ -63,7 +63,7 @@ Experience the future of data storage with 5GB of free, decentralized storage on | |||
|
|||
[Get your free licence here!](https://metaprovide.org/hejbit/start) . | |||
]]></description> | |||
<version>0.5.4</version> | |||
<version>0.5.0</version> |
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.
Is there any reason to update the version like this for this PR?
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.
The version should be higher than the lastest that we publish. I think @frozenbanana controls the app versionning on the release process.
->addIdentifierAlias('\OC\Files\External_Storage\BeeSwarm') // legacy compat | ||
->setStorageClass('\OCA\Files_External_Ethswarm\Storage\BeeSwarm') | ||
->setText($l->t('Swarm By Hejbit')) | ||
->setText($l->t('Swarm')) |
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.
I think @JoaoSRaposo wanted to have the HejBit in name in the settings page.
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.
Yes :)
@@ -425,16 +445,6 @@ public function getMetaData($path) | |||
return $data; | |||
} | |||
|
|||
public function getDirectoryContent($path): \Traversable |
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.
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.
I think we should mantain @Thesola10 version on this one
@@ -104,7 +104,7 @@ private function uploadStream(string $path, string $tmpfile, string $mimetype, i | |||
*/ | |||
private function downloadStream(string $reference) | |||
{ | |||
$endpoint = $this->api_url . DIRECTORY_SEPARATOR . 'bzz' . DIRECTORY_SEPARATOR . $reference . DIRECTORY_SEPARATOR; |
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.
This is also related to #63 and not be reverted.
Enables notifications when files are uploaded (aka decentralized) to Swarm.
The message is set in the code in a new class: /lib/Notifications/Notifier.php.
The notification is sent to the user who uploads the content.
Added the Appname 'files_external_ethswarm' as a constant in a new class for application-wide use.