Skip to content

Conversation

@sidharth-chauhan
Copy link

Problem

The examples/demo, examples/simple, and examples/tutorial in react-admin did not support Docker, requiring users to manually install dependencies and configure each project. This was inconvenient, especially for DevOps users or beginners.

Solution

I have added Docker support and documentation to all three examples:

  • ✅ Added Dockerfile for each example
  • ✅ Added a README.md explaining how to build and run the example using Docker

This makes it easy for anyone to quickly run any example in a containerized environment.

Changes Summary

  • examples/demo/Dockerfile
  • examples/demo/README.md
  • examples/simple/Dockerfile
  • examples/simple/README.md
  • examples/tutorial/Dockerfile
  • examples/tutorial/README.md

Screenshots

Simple App Screenshot 2025-06-06 at 5 39 02 PM Screenshot 2025-06-06 at 5 39 53 PM
Demo App Screenshot 2025-06-06 at 5 42 44 PM Screenshot 2025-06-06 at 5 43 17 PM
Tutorial App Screenshot 2025-06-06 at 5 44 14 PM Screenshot 2025-06-06 at 5 45 04 PM

Checklist

  • All Dockerfiles were tested locally and run successfully
  • Added corresponding README files for Docker usage
  • Follows contribution guidelines
  • Screenshots attached for verification

@sidharth-chauhan
Copy link
Author

“Hi! I’ve added Docker support to the examples. Would love your feedback whenever convenient. Thanks!”

@slax57
Copy link
Contributor

slax57 commented Jun 10, 2025

Hi,
Thank you for your contribution.
IMHO it's pretty straightforward to create docker containers to run react-admin code, as it's a standard node.js application, so I'm not sure this requires specific config files + documentation.
On the other hand this adds code to the codebase that we won't use ourselves but will need to maintain, so I'm -1 for it.
Hence I'm closing this PR.
Thank you anyway for the contribution, and feel free to contribute to other parts of the repo.

@slax57 slax57 closed this Jun 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants