Skip to content

Conversation

@nikgraf
Copy link
Collaborator

@nikgraf nikgraf commented Jan 14, 2026

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Space querying functionality to work with a new API that uses UUIDs for account identification instead of Ethereum addresses. The changes migrate from accountAddress to accountId/memberId/editorId parameters across the codebase.

Changes:

  • Updated GraphQL queries to use accountId (UUID type) instead of accountAddress (String type)
  • Renamed API parameters from memberAccountAddress/editorAccountAddress to memberId/editorId
  • Updated documentation and example code to reflect the new parameter names

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
packages/hypergraph/src/space/find-many-public.ts Updated GraphQL queries, type definitions, and filter parameters to use UUID-based IDs instead of addresses
packages/hypergraph-react/src/hooks/use-spaces.ts Updated GraphQL query parameter from accountAddress to memberId
docs/docs/query-public-data.md Updated documentation examples to use new memberId and editorId parameters
apps/events/src/routes/podcasts.lazy.tsx Updated filter parameter to use memberId with a UUID value and reordered imports
apps/events/src/routes/index.tsx Added test usage of usePublicSpaces with editorId filter and reordered imports
apps/events/src/Boot.tsx Added apiOrigin configuration for testnet API endpoint
.changeset/eighty-hornets-kiss.md Added changeset file documenting the fix

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nikgraf nikgraf merged commit 8a09ffd into main Jan 14, 2026
6 checks passed
@nikgraf nikgraf deleted the nik/fix-fetching-spaces branch January 14, 2026 12:25
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.

2 participants