Skip to content

Testing#1

Draft
ericgaspar wants to merge 15 commits into
masterfrom
testing
Draft

Testing#1
ericgaspar wants to merge 15 commits into
masterfrom
testing

Conversation

@ericgaspar
Copy link
Copy Markdown
Member

No description provided.

@ericgaspar
Copy link
Copy Markdown
Member Author

!testme

@yunohost-bot
Copy link
Copy Markdown
Contributor

Test it, book it, worm it, technologic. 🤖 🎶
Test Badge

@yunohost-bot
Copy link
Copy Markdown
Contributor

Living in the past, are we? 🐮👁️
Test Badge

@ericgaspar
Copy link
Copy Markdown
Member Author

!testme

@yunohost-bot
Copy link
Copy Markdown
Contributor

🚀
Test Badge

@ericgaspar
Copy link
Copy Markdown
Member Author

ericgaspar commented Mar 21, 2026

@otm33GH I'm not really sure that proposing the choice at installation for the storage driver is a good idea (how is the driver configured...).

@otm33GH
Copy link
Copy Markdown
Collaborator

otm33GH commented Mar 21, 2026

@ericgaspar Ok, then it can be removed from manifest and local set as the default storage driver. Let's keep this option in the config panel : wdyt ? I don't have write permissions.

@ericgaspar
Copy link
Copy Markdown
Member Author

!testme

@yunohost-bot
Copy link
Copy Markdown
Contributor

Test it, book it, worm it, technologic. 🤖 🎶
Test Badge

Copy link
Copy Markdown
Collaborator

@otm33GH otm33GH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BTW, do you think features under paid license should be declared or labeled as antifeatures in the catalog ?

Comment thread conf/.env
# options: smtp | postmark
MAIL_DRIVER=smtp
MAIL_FROM_ADDRESS=__MAIL_USER__@__DOMAIN__
MAIL_FROM_ADDRESS=docmost@__DOMAIN__
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TBH it was my first intention but yesterday we talked about that on matrix
https://matrix.to/#/!PauySEslPVuJCJCwlZ:matrix.org/$QQk_m_6PZGIRGcDgSsRRU8YdC8vztmWhDRpepeEGczU?via=matrix.org
What do you think ?

Comment thread config_panel.toml
name.en = "S3 storage"
name.fr = "Stockage S3"

[main.storage_s3.storage_driver]
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mmh... Not very sure to understand this... 🤔 local is not a S3 storage and that's redundant with previous section. Or maybe only display S3 section if S3 is selected ?

@otm33GH otm33GH marked this pull request as draft March 30, 2026 15:22
@otm33GH
Copy link
Copy Markdown
Collaborator

otm33GH commented Mar 30, 2026

Unable to build without ee folders...
See : https://github.com/docmost/docmost?tab=readme-ov-file#license
@ericgaspar I think we cannot package this app since some folders are under paid license and cannot be removed. Am I right?

@Thovi98
Copy link
Copy Markdown
Member

Thovi98 commented Apr 2, 2026

I think you are right if we cannot build the app without ee we cannot propose it in production

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants