feat: add react-admin template with ra-appwrite integration#12
Merged
vermakhushboo merged 1 commit intoJun 19, 2025
Merged
Conversation
There was a problem hiding this comment.
Pull Request Overview
A new React Admin starter template has been added under react/react-admin to bootstrap projects with React Admin and Appwrite integration. This includes project configuration, TypeScript setup, and resource modules for visitors.
- Introduced Vite and TypeScript configuration for the template.
- Scaffolded
visitorsresources (list/create/edit) undersrc/visitors. - Added core resource modules (orders, reviews, products, categories, invoices), layout, themes, and i18n files.
Reviewed Changes
Copilot reviewed 101 out of 105 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| vite.config.ts | Vite configuration with React plugin and build options |
| tsconfig.node.json | Node-specific TypeScript config for vite.config.ts |
| tsconfig.json | Main TypeScript config for the React Admin source |
| src/visitors/segments.ts | Defines segment IDs, currently unused by the template |
| src/visitors/index.ts | Exports Visitor resource definition |
Comments suppressed due to low confidence (2)
react/react-admin/src/visitors/segments.ts:1
- The
segments.tsfile insrc/visitorsis not imported or used by any component—segment data is loaded fromsrc/segments/data.tsinstead. Consider removing this file or consolidating its data with the existingsegments/data.tsto avoid duplication.
const segments = [
react/react-admin/src/visitors/index.ts:1
- [nitpick] Verify that all imports in this file correspond to actual dependencies; the
VisitorIconimport is correct, but review the resource exports to ensure thesegments.tsfile is not intended here, as it's currently unused.
import VisitorIcon from '@mui/icons-material/People';
| @@ -0,0 +1,17 @@ | |||
| const segments = [ | |||
Contributor
There was a problem hiding this comment.
Could you point me to where this is used?
Contributor
There was a problem hiding this comment.
Indeed this file seems unused, nice catch!
Fixing upstream: marmelab/react-admin#10792
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Added a new React Admin starter template, which uses
ra-appwriteusing correct folder structureframework/productto help developers quickly bootstrap projects with React Admin Template. This template provides a ready-to-use setup.Test Plan
The template has been tested locally to make sure:
Related PRs and Issues
No related PRs or issues at this time.
Have you read the Contributing Guidelines on issues?
Yes