Skip to content

Added:SVG-Optimizer#1666

Open
Abhrxdip wants to merge 5 commits intoreactplay:mainfrom
Abhrxdip:main
Open

Added:SVG-Optimizer#1666
Abhrxdip wants to merge 5 commits intoreactplay:mainfrom
Abhrxdip:main

Conversation

@Abhrxdip
Copy link
Contributor

@Abhrxdip Abhrxdip commented Feb 7, 2026

Features Implemented:
✅ Functional components with React Hooks (useState, useEffect)
✅ Controlled inputs for all form elements
✅ Conditional rendering throughout
✅ Client-side processing (no backend needed)
✅ Reusable component architecture
✅ File upload support
✅ Copy to clipboard
✅ Download optimized SVG
✅ Real-time optimization with live preview
✅ File size reduction tracking

@netlify
Copy link

netlify bot commented Feb 7, 2026

Deploy Preview for reactplayio failed. Why did it fail? →

Name Link
🔨 Latest commit d968030
🔍 Latest deploy log https://app.netlify.com/projects/reactplayio/deploys/6989931c3b6c960008a0a060

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hey! contributor, thank you for opening a Pull Request 🎉.

@reactplay/maintainers will review your submission soon and give you helpful feedback. If you're interested in continuing your contributions to open source and want to be a part of a welcoming and fantastic community, we invite you to join our ReactPlay Discord Community.
Show your support by starring ⭐ this repository. Thank you and we appreciate your contribution to open source!
Stale Marking : After 30 days of inactivity this issue/PR will be marked as stale issue/PR and it will be closed and locked in 7 days if no further activity occurs.

Copy link
Member

@priyankarpal priyankarpal left a comment

Choose a reason for hiding this comment

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

The Netlify deploy errored, with the following guidance provided:

Diagnosis

  • #L57 shows that pnpm install stops because json-graphql-parser@0.1.9 has a peer dependency on axios@^0.27.2, while the lockfile currently provides axios@1.13.4. With PNPM’s strict peer-dependency checks enabled on Netlify, this mismatch causes the install (and therefore the build) to fail before the React app can be built.

Solution

  1. Verify that the intended axios version is declared in package.json and that the corresponding pnpm-lock.yaml entry has been committed.
  2. Bring the dependency tree back into compliance:
    • Preferred: upgrade to a version of json-graphql-parser (or an alternative package) that supports axios@1.x, or, if the project can work with the older API, change the declared axios version to ^0.27.2, run pnpm install, and commit the updated lockfile.
    • If axios@1.x must remain for the app, add an override so only json-graphql-parser receives the legacy peer version:
      {
        "pnpm": {
          "overrides": {
            "json-graphql-parser@0.1.9": {
              "axios": "^0.27.2"
            }
          }
        }
      }
      Afterwards, run pnpm install locally and commit the refreshed pnpm-lock.yaml.
  3. Redeploy on Netlify once the peer dependency mismatch has been resolved.

The relevant error logs are:

Line 37: ╰───────────────────────────────────────────────╯
Line 38:  WARN  deprecated eslint@8.57.1: This version is no longer supported. Please see https://eslint.org/version-support for other op
Line 39:  WARN  deprecated @nhost/react@0.9.2: ⚠ DEPRECATED: This package is deprecated in favor of @nhost/nhost-js@^4.0.0. The new SDK i
Line 40:  WARN  deprecated react-p5@1.4.1: Package no longer supported. Contact Support at https://www.npmjs.com/support for more info.
Line 41: Progress: resolved 79, reused 0, downloaded 79, added 0
Line 42:  WARN  deprecated workbox-cacheable-response@6.6.1: workbox-background-sync@6.6.1
Line 43: Progress: resolved 113, reused 0, downloaded 113, added 0
Line 44: Progress: resolved 350, reused 0, downloaded 344, added 0
Line 45: Progress: resolved 821, reused 0, downloaded 788, added 0
Line 46: Progress: resolved 1370, reused 0, downloaded 1338, added 0
Line 47:  WARN  33 deprecated subdependencies found: @babel/plugin-proposal-class-properties@7.18.6, @babel/plugin-proposal-nullish-coale
Line 48: Progress: resolved 1790, reused 0, downloaded 1758, added 0
Line 49: Already up to date
Line 50: Progress: resolved 1790, reused 0, downloaded 1758, added 0, done
Line 51: > react-play@1.0.0 prepare /opt/build/repo
Line 52: > husky install
Line 53: husky - Git hooks installed
Line 54:  WARN  Issues with peer dependencies found
Line 55: .
Line 56: ├─┬ json-graphql-parser 0.1.9
Line 57: │ └── ✕ unmet peer axios@^0.27.2: found 1.13.4

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