Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed errors for stencil #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

codepool867
Copy link

No description provided.

@calvinckho
Copy link

I have run into the same LAME not defined error. This PR fixes the error. Thanks @nikolaytoplev.

This was referenced Apr 21, 2021
@buzinas
Copy link
Contributor

buzinas commented Apr 21, 2021

This file is generated by our build process. If we merge this, as soon as we build/deploy a new version, this will get erased.

@kazzkiq
Copy link

kazzkiq commented Jul 22, 2021

I had to copy this built file and paste it directly into my project in order for it to work.

Not a great solution since I lose all the npm package benefits, but it works until this lib works out how to handle the issue.

@ro-mgh
Copy link

ro-mgh commented Mar 21, 2022

Please merge this))

@vhd-jake
Copy link

vhd-jake commented Jun 9, 2022

Please fix this issue 😢

@vhd-luke
Copy link

vhd-luke commented Jun 9, 2022

Please merge this

1 similar comment
@aleterlone
Copy link

Please merge this

@ianhenrysmith
Copy link

ianhenrysmith commented Aug 1, 2023

Can confirm that this change did fix my issue (ReferenceError: Lame is not defined when using Vite).

@chetan-zdev
Copy link

Can we pls merge this if this fix is working fine

Copy link

@prasoonthakur8 prasoonthakur8 left a comment

Choose a reason for hiding this comment

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

Please approve this

@prasoonthakur8
Copy link

Untill admin approves changes, you can use
npm i mic-recorder-to-mp3-fixed

and can be used as:
import MicRecorder from 'mic-recorder-to-mp3-fixed';

I have merged the 32 in this package.

Thanks

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.