Repository navigation
Conversation
…n time windows Widgets titled YTD / last-12-months / QTD declared their own close_date filters, but the dashboard-level dateRange (default this_quarter) was injected on top, silently intersecting the two. Executive YTD revenue showed 781,012 (quarter) instead of 2,618,600 (year). Adds filterBindings dateRange opt-outs across 4 dashboards (8 widgets) plus a metadata guard test (red on all 8 before the fix, 68 green after). Closes #1 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
New contract_board kanban grouped by status (draft / in-approval / activated / expired / terminated), cards show contract number, account and end date with status-colored badges reusing the field's own option colors; column headers summarize contract_value. Closes #2 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
New crm_competitor object (threat level with colored badges, win/loss battle cards in Markdown, product and intel-owner fields), two list views (all + high-threat), Sales nav entry, full zh/en/ja/es translations, seed data (4 competitors with Chinese battle cards), and a multi-value competitor lookup on opportunities replacing the hardcoded Competitor A/B/C placeholder options. Known platform limitation recorded in seeds: the seed loader drops multi-value lookup natural-key arrays silently; opportunity links were backfilled directly and the intent arrays are kept in data/index.ts for when the loader supports them. Closes #3 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The LICENSE file said MIT while README.md, all 150+ source file headers, and the marketplace docs (en/zh-Hans/zh-Hant) declared Apache-2.0. Standardize everything on Apache-2.0, the license the project publicly advertises: - LICENSE: replace MIT text with the canonical Apache-2.0 text - CONTRIBUTING.md: contributions are licensed under Apache-2.0 - README.legacy.md: update license badge and license section - objectstack.manifest.json: license field MIT -> Apache-2.0 (published to the marketplace via scripts/publish-marketplace.mjs) - package.json: add "license": "Apache-2.0" (fallback source for the marketplace publish script) - scripts/publish-marketplace.mjs: align file header with the rest of the codebase Closes objectstack-ai#487 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015diSVVVtpbSUSfU8Wpu5Jb
|
@claude is attempting to deploy a commit to the Object Stack Team on Vercel. A member of the Team first needs to authorize it. |
This was referenced Jul 29, 2026
yinlianghui
added a commit
that referenced
this pull request
Jul 30, 2026
Reverts the two feature commits that #532's squash merge carried in alongside the license fix. Both were built as recording aids for a demo video series and were not meant to land upstream: - feat(contract): add status kanban board view (5e41bec) - feat(competitor): add competitor management module (7b68275) The opportunity object returns to its previous hardcoded competitors select; the dashboard dateRange fix from the same squash (#499) stays. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
yinlianghui
added a commit
that referenced
this pull request
Jul 30, 2026
revert: remove demo-only contract kanban and competitor module (#532 follow-up)
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
3 participants
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.
Description
The repository declared conflicting licenses:
LICENSEandobjectstack.manifest.jsonsaid MIT, while the README badge/footer and every source file header said Apache-2.0 — andscripts/publish-marketplace.mjspublished that contradictory pair to the marketplace. This PR standardizes everything on Apache-2.0, the license the project publicly advertises (commit3546eae):LICENSE: replace the MIT text with the canonical Apache-2.0 license textobjectstack.manifest.json:"license"fieldMIT→Apache-2.0(the value published to the marketplace)package.json: add"license": "Apache-2.0"(fallback source for the marketplace publish script)CONTRIBUTING.md: state that contributions are licensed under Apache-2.0README.legacy.md: update the license badge and license sectionscripts/publish-marketplace.mjs: align the file header with the rest of the codebasemain(d0e2765) from the fork and additionally carries three fork-side commits beyond the license fix:7b68275feat(competitor): competitor management module,5e41becfeat(contract): status kanban board view, andefa59abfix(dashboards): dateRange filter opt-outs. Their commit messages reference#1/#2/#3(fork-local issue numbers, unrelated to this repo's #1–#3). If only the license fix is wanted here, the branch should be rebased onto the latestmainwith just3546eaebefore merging.Type of Change
Related Issues
Fixes #487
Changes Made
LICENSEwith the canonical Apache-2.0 license textobjectstack.manifest.jsonlicensefromMITtoApache-2.0and added"license": "Apache-2.0"topackage.jsonso the marketplace publish script has a consistent sourceCONTRIBUTING.md,README.legacy.md, and thescripts/publish-marketplace.mjsfile header to consistently state Apache-2.0Testing
npm test) — all tests pass on the head branchnpm run lint)npm run build)Screenshots
N/A — license/documentation changes only.
Checklist
pnpm changeset) — required on every PR, or theskip-changesetlabel is appliedAdditional Notes
skip-changesetlabel may be appropriate; alternatively the head branch predates the new changeset CI requirement (0437c9f) and will need one (or the label) to pass CI.3546eaeif maintainers prefer a minimal diff.🤖 Generated with Claude Code
https://claude.ai/code/session_01JmS4beF8foorE7ykRdKcLU
Generated by Claude Code