Skip to content

add: new sounds added#104

Merged
Utkarshn10 merged 2 commits intoUtkarshn10:masterfrom
KSSaiTeja:add/new-sounds-added
Aug 31, 2023
Merged

add: new sounds added#104
Utkarshn10 merged 2 commits intoUtkarshn10:masterfrom
KSSaiTeja:add/new-sounds-added

Conversation

@KSSaiTeja
Copy link
Copy Markdown
Contributor

Some new sounds added.. please check.. let me know if they're ok.. i will raise another PR.. by adding those sounds

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 29, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @Utkarshn10 on Vercel.

@Utkarshn10 first needs to authorize it.

Copy link
Copy Markdown
Owner

@Utkarshn10 Utkarshn10 left a 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.

@KSSaiTeja
Copy link
Copy Markdown
Contributor Author

@Utkarshn10 i have changed the files names.. please check and let me know..

Copy link
Copy Markdown
Owner

@Utkarshn10 Utkarshn10 left a comment

Choose a reason for hiding this comment

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

LGTM. Approved.

@Utkarshn10 Utkarshn10 merged commit 15b7e51 into Utkarshn10:master Aug 31, 2023
@KSSaiTeja
Copy link
Copy Markdown
Contributor Author

Thanks for merging my PR..

should i add those sounds to the focusly

@Utkarshn10
Copy link
Copy Markdown
Owner

Yes please go ahead.

@KSSaiTeja
Copy link
Copy Markdown
Contributor Author

ok then.. i will start adding those sounds under LO-FI tab

@KSSaiTeja
Copy link
Copy Markdown
Contributor Author

i have raised new PR with another branch please check and merge the PR

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