Skip to content

Fix documentation deployment for www.objectui.org#6

Merged
hotlong merged 6 commits intomainfrom
copilot/fix-website-access-issue
Jan 13, 2026
Merged

Fix documentation deployment for www.objectui.org#6
hotlong merged 6 commits intomainfrom
copilot/fix-website-access-issue

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 13, 2026

GitHub Actions deployment was failing, preventing www.objectui.org from serving. Three issues identified:

Changes

  • Fixed pnpm version mismatch: Updated deploy-docs workflow to use pnpm v10 (matching all other workflows) instead of v8, which was incompatible with the project's lockfile version 9.0

  • Added .nojekyll: Created docs/public/.nojekyll to prevent GitHub Pages from running Jekyll processor on VitePress output

  • Fixed repository references: Corrected objectql/object-uiobjectql/objectui across all documentation, README, badges, and links (actual repo has no dash)

The deployment workflow will succeed on merge to main.

Original prompt

www.objectui.org 已经指向 objectql.github.io 但是还是访问不到官网


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 4 commits January 13, 2026 14:48
Co-authored-by: huangyiirene <7665279+huangyiirene@users.noreply.github.com>
Co-authored-by: huangyiirene <7665279+huangyiirene@users.noreply.github.com>
Co-authored-by: huangyiirene <7665279+huangyiirene@users.noreply.github.com>
Co-authored-by: huangyiirene <7665279+huangyiirene@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix access issue for objectui.org pointing to objectql.github.io Fix documentation deployment for www.objectui.org Jan 13, 2026
Copilot AI requested a review from huangyiirene January 13, 2026 15:11
@hotlong
Copy link
Copy Markdown
Contributor

hotlong commented Jan 13, 2026

@copilot 2026-01-13T15:52:06.3277724Z ##[group]Run pnpm install --frozen-lockfile
2026-01-13T15:52:06.3278182Z �[36;1mpnpm install --frozen-lockfile�[0m
2026-01-13T15:52:06.3305682Z shell: /usr/bin/bash -e {0}
2026-01-13T15:52:06.3306033Z env:
2026-01-13T15:52:06.3306334Z PNPM_HOME: /home/runner/setup-pnpm/node_modules/.bin
2026-01-13T15:52:06.3306666Z ##[endgroup]
2026-01-13T15:52:06.6810955Z Scope: all 10 workspace projects
2026-01-13T15:52:06.7672478Z  WARN  Ignoring not compatible lockfile at /home/runner/work/objectui/objectui/pnpm-lock.yaml
2026-01-13T15:52:06.7829580Z  ERR_PNPM_NO_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is absent
2026-01-13T15:52:06.7830145Z
2026-01-13T15:52:06.7830829Z Note that in CI environments this setting is true by default. If you still need to run install in such cases, use "pnpm install --no-frozen-lockfile"
2026-01-13T15:52:06.7996507Z ##[error]Process completed with exit code 1.

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jan 13, 2026

@copilot 2026-01-13T15:52:06.3277724Z ##[group]Run pnpm install --frozen-lockfile
2026-01-13T15:52:06.3278182Z �[36;1mpnpm install --frozen-lockfile�[0m
2026-01-13T15:52:06.3305682Z shell: /usr/bin/ba...

Fixed in commit ec710c8. The issue was a pnpm version mismatch - the deploy-docs workflow was using pnpm v8 while the project requires pnpm v10 (all other workflows were already using v10). Updated the workflow to use pnpm v10 and restored the original lockfile (version 9.0).

@github-actions
Copy link
Copy Markdown
Contributor

✅ All checks passed!

  • ✅ Type check passed
  • ✅ Tests passed
  • ✅ Lint check completed

Copilot AI requested a review from hotlong January 13, 2026 16:03
@hotlong hotlong marked this pull request as ready for review January 13, 2026 16:05
@hotlong hotlong merged commit 1be42ef into main Jan 13, 2026
5 checks passed
Copilot AI added a commit that referenced this pull request Jan 24, 2026
- Fix handleExportCSV to guard on gridRef.current?.api (issue #1)
- Add dedicated onContextMenuAction callback instead of overloading onCellClicked (issue #2)
- Remove icon property from customItems to prevent HTML injection (issue #3)
- Remove validation claim from README - only basic AG Grid editing (issue #4)
- Add test assertions for all new inputs (editable, exportConfig, etc.) (issue #5)
- Fix onExport type to only support 'csv' format (issue #6)
- Remove unused ColumnConfig properties (autoSize, groupable) (issue #9)
- Type schema props with proper interfaces instead of 'any' (issue #10)
- Update export description to only mention CSV (issue #11)
- Add AG Grid Community vs Enterprise section to docs (issue #8)
- Update README and docs with new callback and clarifications

All tests pass (8/8), lint clean (0 errors)

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Copilot AI added a commit that referenced this pull request Feb 20, 2026
…warning, i18n fallback

- Issue #1: Normalize `in`/`not in` operators to backend-compatible `or`/`and` of `=`/`!=`
- Issue #2: Filter merging now validates and filters empty conditions
- Issue #3: CSV export safely serializes arrays (semicolon-separated) and objects (JSON)
- Issue #5: Request counter prevents stale data from overwriting latest results
- Issue #6: PullToRefresh resets pull distance immediately to prevent UI lock
- Issue #7: $top configurable via schema.pagination, data limit warning shown
- Issue #8: Extended i18n fallback translations for all ListView labels
- Issue #9: Defensive null checks in effectiveFields for mismatched objectDef
- Issue #10: Added FilterNormalization, Export, and DataFetch test suites

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
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.

3 participants