Dashboard Phase 1 refresh#31
Draft
Protocol-zero-0 wants to merge 3 commits into
Draft
Conversation
Contributor
Author
|
Follow-up for issue #30 latest comment is implemented on Changed files:
Acceptance results:
Notes:
|
Contributor
Author
|
i18n follow-up completed on Changed files:
Validation:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changed files
web/templates/index.htmlweb/static/js/app.jsweb/static/js/i18n.jsweb/static/css/style.cssweb/app.pytests/test_web_app.pySummary
en/zhdictionaries,navigator.languagedefaulting,localStoragelanguage persistence, anddata-i18nstatic text application._planned_tracksimplementation and the obsolete ChineserenderExperimentGroupsfunction.Acceptance commands and results
.venv/bin/python -m pytest tests/test_web_app.py -q->12 passed in 3.63s.venv/bin/python -c "import web.app; print('import ok')"->import okDEEPGRAPH_DB_PATH=/tmp/deepgraph-dashboard-refresh-test.db FLASK_APP=web.app .venv/bin/flask run --host 127.0.0.1 --port 5099; then:curl -s -o /tmp/deepgraph-root.out -w "%{http_code}" http://127.0.0.1:5099/->200curl -s -o /tmp/deepgraph-stats.out -w "%{http_code}" http://127.0.0.1:5099/api/stats->200rg -n "Paper DB|Pipeline Papers|Paper Ideas|Method x Dataset|Method × Dataset|Taxonomy Map|Opportunity Map|Deep Insight|Deep Insights|IDEA #|RUN #|Main run|Research Paper Generation|Generated Papers|Complete Papers|\bforge\b" web/templates/index.html web/static/js/app.js web/static/js/agenda.js web/static/js/manuscript_routing.js-> no matches.venv/bin/python -m py_compile web/app.py-> passedgit diff --check-> passedRedeploy steps
git fetch && git checkout feat/dashboard-refresh.envand database untouched; existing configuration and data are preserved.sudo systemctl restart deepgraph-webhttps://deepgraph.sora2.todayhas a faster first paint, language switching works, and terminology is clean.Known non-goals