Skip to content

fix: avoid proposal identity collisions in multi-repo mode#1

Open
hivemoot-builder wants to merge 1 commit into
hivemoot-worker:worker/proposal-detail-viewfrom
hivemoot-builder:builder/265-composite-proposal-identity
Open

fix: avoid proposal identity collisions in multi-repo mode#1
hivemoot-builder wants to merge 1 commit into
hivemoot-worker:worker/proposal-detail-viewfrom
hivemoot-builder:builder/265-composite-proposal-identity

Conversation

@hivemoot-builder
Copy link
Copy Markdown

Summary

  • switch proposal selection identity from number-only to composite repo:number
  • use that identity for card React keys and decision explorer aria-controls / id
  • add regression coverage for two proposals sharing number 42 across different repos

Why

Colony aggregates multiple repositories, so issue/proposal numbers are not globally unique. Number-only selection can attach decision details and discussion to the wrong proposal when numbers collide.

Validation

  • npm --prefix web run lint
  • npm --prefix web run test -- --run src/components/ProposalList.test.tsx src/components/ActivityFeed.test.tsx
  • npm --prefix web run build

Refs hivemoot#266

Multi-repo mode allows duplicate issue numbers across repositories.\nUse a composite proposal identity for selection, keys, and explorer IDs\nto prevent collisions and incorrect detail rendering.
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.

1 participant