Skip to content

docs: align README with Render setup and replace Fly mentions - #42

Merged
Ymax27 merged 1 commit into
Ymax27:mainfrom
obithelight:docs/readme-render-deploy
Oct 19, 2026
Merged

Ymax27 merged 1 commit into
Ymax27:mainfrom
obithelight:docs/readme-render-deploy

Conversation

@obithelight

Copy link
Copy Markdown
Contributor

What I changed

Replaced every mention of Fly in README.md with the Render + Neon setup:

  • Stack line
  • OAuth table (heading, homepage URL, callback URL)
  • Production login checklist (items 3, 4 and 5)
  • "Déploiement" section, rewritten for Render + Neon (Blueprint, secret
    variables, OAuth callback, the two crons, pointer to render.yaml)

Written in French to match the rest of the README, as requested.

Why

Closes #3

Production runs on Render (opentoall.onrender.com), but the README still
pointed new contributors to Fly.

How I checked

  • Cross-checked the new deploy section with render.yaml and entrypoint.sh.
  • Used the callback URL given in the issue.
  • Searched the README for "fly": no mention is left.
  • Did not touch CONTRIBUTING.md, so its label links are unchanged.

Questions

  1. The issue gave only the callback URL, so I used
    https://opentoall.onrender.com as the Homepage URL. Is that correct?
  2. I removed fly deploy and fly.toml and did not add a Render equivalent,
    because the issue says not to invent steps. The deploy section points to
    render.yaml instead.
  3. USE_REDIS_CACHE=0 was in the old Fly secrets list but is not in
    render.yaml, so I left it out. Is this okay as per requirements or
    should it still be documented?

@Ymax27

Ymax27 commented Oct 19, 2026

Copy link
Copy Markdown
Owner

Thanks a lot @obithelight, this is a clean first PR and exactly the scope the issue asked for.

  1. Yes, https://opentoall.onrender.com is the right Homepage URL.
  2. Good call. Pointing to render.yaml is what I wanted: the Blueprint handles the deploy, so there's no manual command to document.
  3. Fine to leave it out. USE_REDIS_CACHE already defaults to off when no remote Redis URL is set (see config/settings.py), so Render doesn't need it.

I've approved the CI run and will merge once the tests pass 🙌

— Max

@Ymax27
Ymax27 merged commit 292bc61 into Ymax27:main Oct 19, 2026
1 check passed
@Ymax27

Ymax27 commented Oct 19, 2026

Copy link
Copy Markdown
Owner

Merged 🎉 Congrats on your first contribution to OpenToAll, @obithelight, and thanks for the careful cross-checking with render.yaml. New contributors won't land on Fly anymore. Hope to see you around, there are more good first issues waiting!

— Max

@obithelight

Copy link
Copy Markdown
Contributor Author

Thank you @Ymax27 for the quick review and clear answers. I learned a lot from this. I'll look at the other good first issues.

@obithelight
obithelight deleted the docs/readme-render-deploy branch October 19, 2026 15:06
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.

Align the README deploy section with the Render setup

2 participants