Skip to content

Revert "refactor(deps): upgrade @vocdoni/davinci-sdk"#46

Merged
elboletaire merged 3 commits into
mainfrom
f/downgrade-sdk
Nov 19, 2025
Merged

Revert "refactor(deps): upgrade @vocdoni/davinci-sdk"#46
elboletaire merged 3 commits into
mainfrom
f/downgrade-sdk

Conversation

@elboletaire

Copy link
Copy Markdown
Member

This reverts commits 1cbf572 and 589adde with some manual tweaks due to lots of other changes

This reverts commits 1cbf572 and 589adde with some manual tweaks due to lots of other changes
Copilot AI review requested due to automatic review settings November 19, 2025 09:13

Copilot AI left a comment

Copy link
Copy Markdown

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 reverts a previous upgrade to @vocdoni/davinci-sdk from version 0.0.3 back to 0.0.2. The changes remove SDK-based process creation and event listening functionality, reverting to the older direct API approach for creating voting processes.

Key changes:

  • Downgrade @vocdoni/davinci-sdk from ~0.0.3 to ~0.0.2
  • Remove SDK initialization and event listener setup from VocdoniApiContext
  • Replace SDK-based process creation stream with direct API calls and smart contract interactions
  • Remove useCallback wrappers from several provider functions

Reviewed Changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
package.json Downgrades davinci-sdk version from 0.0.3 to 0.0.2
src/contexts/vocdoni-api-context.tsx Removes SDK instance, simplifies to only expose API service
src/components/create-vote-form.tsx Replaces SDK stream-based process creation with manual API and contract calls
src/hooks/use-unified-provider.ts Removes useCallback wrappers from provider functions
src/contexts/MiniAppContext.tsx Removes useCallback wrapper from getFarcasterEthereumProvider
src/router.tsx Updates destructuring to match new context API
src/hooks/use-vote-status.ts Updates destructuring to match new context API
src/hooks/use-process-query.ts Updates type annotation and destructuring for new context API
src/contexts/election-context.tsx Updates destructuring to match new context API
src/components/vote-display.tsx Updates destructuring to match new context API
src/components/vote-actions.tsx Updates destructuring to match new context API
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)

src/hooks/use-unified-provider.ts:1

  • The React import is unnecessary since no JSX or React types are used in this file. Remove the unused import.
/* eslint-disable @typescript-eslint/no-explicit-any */

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

Comment thread src/components/create-vote-form.tsx
@github-actions

github-actions Bot commented Nov 19, 2025

Copy link
Copy Markdown

@github-actions github-actions Bot temporarily deployed to pull request November 19, 2025 09:14 Inactive
@github-actions github-actions Bot temporarily deployed to pull request November 19, 2025 09:24 Inactive
@github-actions github-actions Bot temporarily deployed to pull request November 19, 2025 09:30 Inactive
@elboletaire elboletaire merged commit 18f2967 into main Nov 19, 2025
2 checks passed
@github-actions github-actions Bot temporarily deployed to production November 19, 2025 10:07 Inactive
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