lokidundun opened a new pull request, #348:
URL: https://github.com/apache/hugegraph-computer/pull/348
<!--
Thank you very much for contributing to Apache HugeGraph, we are happy
that you want to help us improve it!
Here are some tips for you:
1. If this is your first time, please read the [contributing
guidelines](https://github.com/apache/hugegraph/blob/master/CONTRIBUTING.md)
2. If a PR will fix/close a issue, type the message "close xxx" (xxx is
the link of related issue) in the content, github will auto link it (Required)
3. Name the PR title in "Google Commit Format", start with "feat | fix |
perf | refactor | doc | chore",
such like: "feat(core): support the PageRank algorithm" or "fix: wrong
break in the compute loop" (module is optional)
skip it if you are unsure about which is the best component.
4. One PR address one issue, better not to mix up multiple issues.
5. Put an `x` in the `[ ]` to mark the item as CHECKED. `[x]` (or click
it directly after published)
-->
## Purpose of the PR
- close #331 <!-- or use "fix #xxx", "xxx" is the ID-link of related issue,
e.g: close #257 -->
<!--
Please explain more context in this section, clarify why the changes are
needed.
For example:
- If you propose a new API, clarify the use case for a new API.
- If you fix a bug, you can clarify why it is a bug, and should associated
with an issue.
-->
## Main Changes
<!-- Please clarify what changes you are proposing. The purpose of this
section is to outline the changes and how this PR fixes the issue. These change
logs are helpful for better ant faster reviews.)
For example:
- If you introduce a new feature, please show detailed design here or add
the link of design documentation.
- If you refactor some codes with changing classes, showing the class
hierarchy will help reviewers.
- If there is a discussion in the mailing list, please add the link. -->
Build System
- Add scripts/download_ui_assets.sh to download jQuery, Bootstrap, and
Glyphicons at build time
- Add ui/package.json for frontend dependency management
- Update Makefile with download-ui-assets target and integrate it into
init and generate-assets workflows
- Update build.sh to automatically download UI assets before building
- Update Dockerfile to install npm/bash/curl and download UI assets during
image build
Asset Path Restructure
- Change asset path from ui/ to ui/ui/ in asset.go and asset_dev_ui.go to
match the new directory layout
Git/Docker Ignore Rules
- Replace wildcard .dockerignore patterns with explicit file/directory
exclusions
- Add UI dependency patterns to .gitignore (node_modules, lib/* except
functions.js and vermeer.css)
Configuration
- Remove license header from worker.ini template
- Change default master_peer to Docker network IP (172.25.0.10:6689)
- Add auth and auth_token_factor fields to worker.ini
- Add docker-compose.yml for local development with auth enabled
Documentation
- Update README.md with:
- Expanded directory structure showing UI files
- Node.js/npm as a prerequisite
- Authentication configuration and token generation guide
- Web UI access notes requiring auth=token
- Updated monitoring section URL (/ui/master.html)
Result
<img width="2548" height="1041"
alt="img_v3_02121_4371f1c2-2f5f-4020-ab25-a62630c6a0fg"
src="https://github.com/user-attachments/assets/e8e193a0-e3b0-49f5-a0a6-53cbb62c9df1";
/>
## Verifying these changes
<!-- Please pick the proper options below -->
- [ ] Trivial rework / code cleanup without any test coverage. (No Need)
- [ ] Already covered by existing tests, such as *(please modify tests
here)*.
- [ ] Need tests and can be verified as follows.
<!-- Please provide more details about verification
For example:
- If you test manually, please provide related screenshot.
-->
## Does this PR potentially affect the following parts?
<!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
- [ ] Nope
- [ ] Dependencies (add/update license info) <!-- Don't forget to
add/update the info in "LICENSE" & "NOTICE" files (both in root & dist module)
-->
- [ ] Modify configurations
- [ ] The public API
- [ ] Other affects (typed here)
## Documentation Status
<!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
- [ ] `Doc - TODO` <!-- Your PR changes impact docs and you will update
later -->
- [ ] `Doc - Done` <!-- Related docs have been already added or updated -->
- [ ] `Doc - No Need` <!-- Your PR changes don't impact/need docs -->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
