Rename songs array to audio array in Config #503
danpastori
started this conversation in
Ideas
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
👉 Describe the problem
Right now you add audio to the AmplitudeJS configuration through a
songs
array. This doesn't make sense if you are working with audio files that aren't songs such as podcasts.👥 Problem evidence & reach
Any user that works with
podcasts
,interviews
, etc.🥰 Describe the "impact" on users?
It's more developer friendly to name the
songs
array asaudio
if the user is working with audio files that aren't songs (i.e podcasts).🏆 How to solve this problem
Rename the
songs
array toaudio
and update all references within the code. This will be a breaking change from 5.3 -> 6.0 but will document it.💯 How do we validate the problem is solved?
Users can semantically set
audio
instead ofsongs
within AmplitudeJS making it more inline with their use case.Beta Was this translation helpful? Give feedback.
All reactions