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

Added loginPage #1049

Open
wants to merge 65 commits into
base: main
Choose a base branch
from
Open

Added loginPage #1049

wants to merge 65 commits into from

Conversation

choizhang
Copy link

@choizhang choizhang commented Mar 11, 2025

Description

Added a login page, optional through .evn variable settings

Related Issues

[Reference any related issues or tasks addressed by this pull request.]
#1000
#1039

Checklist

  • Changes tested locally
  • Code reviewed
  • Documentation updated (if necessary)
  • Unit tests added (if applicable)

Additional Notes

[Add any additional notes or context for the reviewer(s).]

choizhang and others added 30 commits March 11, 2025 14:48
- Create SettingsDisplay component
- Display graphQueryMaxDepth and graphMinDegree
- Position display at bottom-left corner
- Remove redundant label caching logic
- Add graphLabels state to graph store
- Raised max layout iterations to 30
- Set default iterations to 15
- Move i18n initialization to async function
- Sync i18n with settings store language
- Add Root component for i18n loading state
- Convert i18n.js to TypeScript
- Optimize graph data fetching conditions
- Add isFetching state to prevent duplicate requests
- Improve label selection handling
- Add NoCacheStaticFiles class
- Set Cache-Control headers for HTML files
- Add `t` to `GraphLabels` dependency array
- Add `isFetching` to `useLightragGraph` dependencies
- Updated settings version and added new graph features
- Added enableNodeDrag setting
- Added enableHideUnselectedEdges
- Disabled edge events by default
- Set graphMinDegree to 0
- Set OpenAPI schema URL to `/openapi.json`
- Set docs URL to `/docs`
- Set redoc URL to `/redoc`
- Update Vite config for API docs routing
- Ensure proper path handling for docs endpoints
- Introduced TabVisibilityProvider component
- Created TabContent for conditional rendering
- Added context and hooks for tab visibility
- Updated DocumentManager dependencies
- Integrated provider in App component
@danielaskdd danielaskdd self-assigned this Mar 13, 2025
@danielaskdd
Copy link
Collaborator

danielaskdd commented Mar 14, 2025

Welcome to the LightRAG community, and thanks for your contribution. The landing page is indeed a much-needed functionality. Since we have made several optimizations and evolutions to our WebUI, with some restructuring of the underlying architecture, certain features (such as language selection) overlap with what you have developed. Therefore, before we proceed with further review of your code, we need you to merge the improve-property-tooltip branch from the repository, resolve any conflicts, make appropriate integrations, and then submit again. Once again, thank you for your contribution to the community.

@danielaskdd danielaskdd marked this pull request as draft March 14, 2025 03:03
@danielaskdd danielaskdd added ui web ui good first issue Good for newcomers labels Mar 14, 2025
@danielaskdd
Copy link
Collaborator

You can refer to #1064 for what improve-property-tooltip is about. Any recommendation is welcome.

@choizhang choizhang marked this pull request as ready for review March 14, 2025 16:54
@choizhang
Copy link
Author

As you said, all the work has been completed. I see that you have added a settings dropdown menu, which is indeed a better interaction when it comes to themes in more than 2 languages. Additionally, I have added an exit button

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers ui web ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants