assemblee-virtuelle / assemblee-virtuelle/semapps

Remove react-admin dependency from data provider

Open
#1,221 4 comments 2 reactions 0 assignees View on GitHub
data-provider refactoring
Dominant language
TypeScript
Stars
103
Forks
14
Avg merge
1m
Merged PRs (30d)
2

Description

In order to use the data provider with [Astro](https://astro.build), we should remove `react-admin` and `react` from the peerDependencies of the package.

- Create a local copy of react-admin `fetchUtils.fetchJson`
- Add `react-admin` to the devDependencies so that it can be used to import Typescript declarations ?
- Put hooks into a `@semapps/hooks` packages ?
- Remove reification components or move them to `@semapps/input-components` ?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the data provider package and its peerDependencies, then inspect its use of react-admin's fetchUtils.fetchJson, hooks, and reification components. Check how @semapps/input-components and the proposed @semapps/hooks package relate to those pieces. Done means the data provider can be used with Astro without react-admin or react peer dependencies, with the remaining component and type needs resolved.

Written by the indexing model from the issue text.

Assessment

Tech stack
react, typescript
Domain
frontend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.