-
Notifications
You must be signed in to change notification settings - Fork 44
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
add: new sounds added #104
Conversation
Someone is attempting to deploy a commit to a Personal Account owned by @Utkarshn10 on Vercel. @Utkarshn10 first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @KSSaiTeja please name the sounds based on the names you are going to keep in cards.
@Utkarshn10 i have changed the files names.. please check and let me know.. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Approved.
Thanks for merging my PR.. should i add those sounds to the focusly |
Yes please go ahead. |
ok then.. i will start adding those sounds under LO-FI tab |
i have raised new PR with another branch please check and merge the PR |
Some new sounds added.. please check.. let me know if they're ok.. i will raise another PR.. by adding those sounds