lorawan-stack
lorawan-stack copied to clipboard
Remove `api` usage in UI components
Summary
We should remove usage of the api module in UI components.
Why do we need this?
Separate UI and api usage.
What is already there? What do you see now?
api used via redux middleware (preferred)
api passed via redux connect function
api used directly in UI compoennts
What is missing? What do you want to see?
api is still used in react components:
- Application downlink formatter - https://github.com/TheThingsNetwork/lorawan-stack/pull/5499
- Application uplink formatter - https://github.com/TheThingsNetwork/lorawan-stack/pull/5499
- Device importer - https://github.com/TheThingsNetwork/lorawan-stack/pull/5499
- Application add - https://github.com/TheThingsNetwork/lorawan-stack/pull/5499
- Application api key add - https://github.com/TheThingsNetwork/lorawan-stack/pull/3452
- Application api key edit - https://github.com/TheThingsNetwork/lorawan-stack/pull/3452
- Application collaborator add - https://github.com/TheThingsNetwork/lorawan-stack/pull/3443
- Application collaborator edit - https://github.com/TheThingsNetwork/lorawan-stack/pull/3443
- Application mqtt integration - https://github.com/TheThingsNetwork/lorawan-stack/pull/5499
- Application pub/sub integration add - https://github.com/TheThingsNetwork/lorawan-stack/pull/5499
- Application pub/sub integration edit - https://github.com/TheThingsNetwork/lorawan-stack/pull/5499
- Device add - https://github.com/TheThingsNetwork/lorawan-stack/pull/5499
- Gateway add - https://github.com/TheThingsNetwork/lorawan-stack/pull/5499
- Gateway api key add - https://github.com/TheThingsNetwork/lorawan-stack/pull/3555
- Gateway api key edit - https://github.com/TheThingsNetwork/lorawan-stack/pull/3555
- Gateway collaborator add
- Gateway collaborator edit
- Create account
- Forgot password
- Login
- Update password
- Validate
How do you propose to implement this?
Move api usage to redux middleware or pass api functions via connect.
Can you do this yourself and submit a Pull Request?
yes