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

Howler #41

Merged
merged 4 commits into from
Feb 11, 2025
Merged

Howler #41

merged 4 commits into from
Feb 11, 2025

Conversation

jasonyu0705
Copy link
Collaborator

Developer: Jason Yu, Derrick Phan

Closes #25

Pull Request Summary

Finishing the integration with Howler.js

Modifications

audioplayer.tsx: integrating audio file component in react
route.ts in API: allows communication with mongoDB
audiofile schema: mongo schema with all the properties of the audio file

NOTE: there is some test code in the main page.tsx file

Testing Considerations

tested that component shows, should make sure that it is properly connect with mongo

Pull Request Checklist

  • Code is neat, readable, and works
  • Comments are appropriate
  • The commit messages follows our guidelines
  • The developer name is specified
  • The summary is completed
  • Assign reviewers

Screenshots/Screencast

image looks sorta meh bc there is no css atm {put screenshots of your change, or even better a screencast displaying the functionality}

Copy link

vercel bot commented Feb 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
lcslo-kco ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 11, 2025 3:54am

@gracebw7 gracebw7 self-requested a review February 11, 2025 03:18
@gracebw7 gracebw7 marked this pull request as draft February 11, 2025 03:51
@gracebw7 gracebw7 marked this pull request as ready for review February 11, 2025 03:52
@gracebw7 gracebw7 merged commit e4f20b4 into develop Feb 11, 2025
2 of 6 checks passed
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.

Integrating Howler.js
3 participants