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

feat: add soundcloud embed #175

Merged

Conversation

Cahllagerfeld
Copy link
Contributor

@Cahllagerfeld Cahllagerfeld commented May 3, 2022

closes #174

Hey Scott 👋 First of all I really like the idea of your package.
I'm having the following two questions

  • Which Soundcloud-Track do you want to put to the index-page as showcase?
  • The Soundcloud API for embeds is much more complex than the options I added currently (see Widget-API). Do you want to fully support them, or leave it simple for now?

In case you aren't interested in adding SoundCloud, feel free to close the PR 👍 🛹

@vercel
Copy link

vercel bot commented May 3, 2022

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

Name Status Preview Updated
sveltekit-embed ✅ Ready (Inspect) Visit Preview May 3, 2022 at 0:13AM (UTC)

@spences10 spences10 marked this pull request as ready for review May 3, 2022 14:00
@spences10
Copy link
Owner

Awesome! Thanks

@spences10 spences10 merged commit d512cfd into spences10:main May 3, 2022
@spences10
Copy link
Owner

Oh! Hey, I need to add some contributing docs actually!

Would you like to add an example to the src/routes/index.svelte file?

@Cahllagerfeld
Copy link
Contributor Author

Sure :D That's one question I had: which Soundcloud track do you want to showcase on the index page? :D

@spences10
Copy link
Owner

Yeah, sure, I was just adding in the example actually!

Regarding the rest of the configuration, awesome that SoundCloud have a Widget-API!

Feel free to contribute more if you like, I'd appreciate it for sure!

I've added in an example using the component on the index for now, if you want to add more feel free, thanks.

I'm just about to publish your change now.

@spences10
Copy link
Owner

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.

[Feat] Add Embed for SoundCloud
2 participants