Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add frontend support for server-side settings storage #5759

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

malhotra5
Copy link
Contributor

@malhotra5 malhotra5 commented Dec 23, 2024

This PR adds frontend support for the server-side settings storage feature introduced in #5594.

Changes

  • Add Settings interface and API endpoints for loading/storing settings
  • Create settings slice for Redux state management
  • Add useSettings hook for easy access to settings functionality
  • Update conversation initialization to use stored settings

This PR complements #5594 by providing the frontend implementation needed to persist user settings (including API keys) on the server side.

Testing

  • Test loading settings from backend
  • Test saving settings to backend
  • Test settings persistence across sessions
  • Test merging stored settings with conversation overrides

To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:b3ad9df-nikolaik   --name openhands-app-b3ad9df   docker.all-hands.dev/all-hands-ai/openhands:b3ad9df

openhands-agent and others added 8 commits December 23, 2024 04:50
- Add Settings interface and API endpoints
- Create settings slice for Redux state management
- Add useSettings hook for easy access to settings functionality
- Update conversation initialization to use stored settings
- Remove settings version and migration code (no longer needed with backend storage)
- Update settings service to use backend API instead of localStorage
- Update settings utils to use snake_case keys
- Update account settings form to use new settings format
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.

2 participants