Skip to content

Conversation

@wowi42
Copy link
Contributor

@wowi42 wowi42 commented Jul 18, 2025

Describe your PR

Create a new guide for send, an alternative to WeTransfer

@wowi42 wowi42 changed the title Create a new guide for send, an alternative to WeTranfer Create a new guide for send, an alternative to WeTransfer Jul 18, 2025
@davlgd davlgd requested a review from Copilot July 18, 2025 09:48
@davlgd davlgd self-assigned this Jul 18, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR creates a comprehensive deployment guide for Send, a privacy-focused file sharing application that serves as an alternative to WeTransfer. The guide provides step-by-step instructions for deploying Send on Clever Cloud platform with Redis and S3-compatible storage.

  • Complete deployment walkthrough including prerequisites, installation, and configuration
  • Integration setup for Redis metadata storage and Cellar S3 file storage
  • Configuration examples for environment variables, custom branding, and domain setup

@@ -0,0 +1,248 @@
---
title: 'Send'
description:
Copy link

Copilot AI Jul 18, 2025

Choose a reason for hiding this comment

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

The description field is empty. Add a meaningful description that summarizes what this guide covers, such as 'A comprehensive guide to deploying Send, a secure file sharing application, on Clever Cloud platform.'

Suggested change
description:
description: 'A comprehensive guide to deploying Send, a secure file-sharing application, on the Clever Cloud platform.'

Copilot uses AI. Check for mistakes.
tags:
- guides
keywords:
-
Copy link

Copilot AI Jul 18, 2025

Choose a reason for hiding this comment

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

The keywords field contains only a dash. Add relevant keywords like 'send', 'file-sharing', 'privacy', 'wetransfer-alternative', 'clever-cloud' to improve discoverability.

Suggested change
-
- send
- file-sharing
- privacy
- wetransfer-alternative
- clever-cloud

Copilot uses AI. Check for mistakes.
@github-actions
Copy link

github-actions bot commented Jul 18, 2025

✅ Review app deployed

Check how your changes look like!

ℹ️ Name 🔗 Infos & links
🌱 Latest deployed commit 5975e37
👁️ Preview https://documentation-PR-724.cleverapps.io/developers

Copy link
Collaborator

@davlgd davlgd left a comment

Choose a reason for hiding this comment

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

Thanks for this, some changes to the config/scripts before we could merge

@@ -0,0 +1,219 @@
---
title: 'Send'
description: "Simple, private file sharing"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use a longer description (about 100 characters)

- cellar
- s3

draft: false
Copy link
Collaborator

Choose a reason for hiding this comment

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

you can remove this, it's false by default

- s3

draft: false
type: send
Copy link
Collaborator

Choose a reason for hiding this comment

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

Put this as the first parameter of the frontmatter

Suggested change
type: send
type: docs


Before deploying Send on Clever Cloud, make sure you have:

- **Node.js 16.x**
Copy link
Collaborator

Choose a reason for hiding this comment

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

Node.js 16 is EOL

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We need to patch upstream

Copy link
Collaborator

Choose a reason for hiding this comment

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

My point is: it's not the needed version, it's maybe the minimum required version. But if the user can use a supported version, we should recommend it in settings, and here mention that the requirement is 16+. If Node.js 16 is mandatory, we can let it as is, but there are security concerns on non supported releases.

- **Node.js 16.x**
- **Redis** for metadata storage
- **Cellar S3** for file storage
- **Clever Tools CLI** ([documentation](https://www.clever-cloud.com/developers/doc/cli/))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use link relative to the root domain, it's easier to manage when we change domain/path

Suggested change
- **Clever Tools CLI** ([documentation](https://www.clever-cloud.com/developers/doc/cli/))
- **Clever Tools CLI** ([documentation](/developers/doc/cli/))


```bash
# Redis configuration
clever env set REDIS_DB 0
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is it mandatory? It's the default ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Comment on lines +125 to +126
clever env set S3_ENDPOINT <CELLAR_ADDON_HOST>
clever env set AWS_ACCESS_KEY_ID <CELLAR_ADDON_KEY_ID>
clever env set AWS_SECRET_ACCESS_KEY <CELLAR_ADDON_KEY_SECRET>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use a binding script as we did in oTree, you can fuse this with post build commands in the same script

Copy link
Contributor Author

Choose a reason for hiding this comment

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

when you do some otree, you build your app with otree, more like a framework. Here it is not the same: we are using the source code directly from them. I do not want to commit upstream some Clever Cloud specific code.


```bash
# Base URL (replace with your test domain)
clever env set BASE_URL https://<your-test-domain>
Copy link
Collaborator

Choose a reason for hiding this comment

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

you can set it from the application domain (clever domain favourite)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated the documentation to be more clear

clever domain add <your-custom-domain.com>

# Update the BASE_URL environment variable
clever env set BASE_URL https://<your-custom-domain.com>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Already done before

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no, different paragraphs

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we should have one only section for domain configuration. We can tell the user to set a default cleverapps.io or its own domain here, but should't tell him to set one, then the other and restart

clever env set BASE_URL https://<your-custom-domain.com>

# Restart to apply new BASE_URL
clever restart
Copy link
Collaborator

Choose a reason for hiding this comment

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

unnecessary if we do it before

Copy link
Contributor Author

Choose a reason for hiding this comment

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

NA

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.

2 participants