-
Notifications
You must be signed in to change notification settings - Fork 55
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
"Modules & Marketplace" page outdated #79
Comments
@Semir1212 @luke- what do you think about this PR? |
@marc-farre Thank you, @mbumpalumpa will look into it and also provide some screenshots to make it easier for the users. |
What do you think about the new PR @marc-farre? |
@mbumpalumpa thanks, it's awesome! I've added a commit: 9502d7a Because, when you install a new custom module by manually unzipping it, you don't need to perform database migrations (it's done when the module is enabled). So if you agree with my commit, maybe we could remove the screenshot related to DB migrations, or if not, we should add one for flushing cache. But both procedures might change in a future HumHub versions as the Information page might be reworked. |
Both approaches would be fine for me @marc-farre Just pick the one you prefer and adapt the screenshots @mbumpalumpa |
@marc-farre @Semir1212 I've updated it with an additional screenshot and a new screenshot for PE activation. Also, I rewrote the "update" part a bit; let me know if it's okay. |
@mbumpalumpa Thanks, it's perfect! |
On https://docs.humhub.org/docs/admin/modules#purchased-modules we have:
This should be updated, but:
The text was updated successfully, but these errors were encountered: