[D8] Video Embed Dacast

Hi @kimberleycgm,

Thank you for contribution!!!

@kimberleycgm

You can also add composer.json file in module directory.

It's pretty good for me, maybe one thing I would try to change using static \Drupal::configFactory hereto DI, but I know that it's static method so it's impossible use $this there.
By the way I am curious how is the best way to achieve that if you of course have to use a static method.

Dacast::getIdFromInput(): you need to use preg_quote() for the user provided domain name before using it in the regex.

Otherwise looks good to me.

Thanks for your contribution, Kimberley!

I updated your account so you can opt into security advisory coverage now.

Here are some recommended readings to help with excellent maintainership:

• Dries' post on Responsible maintainers
• Best practices for creating and maintaining projects
• Maintaining a drupal.org project with Git
• Commit messages - providing history and credit
• Release naming conventions.
• Helping maintainers in the issue queues

You can find lots more contributors chatting on IRC in #drupal-contribute. So, come hang out and stay involved!

Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

Thanks to the dedicated reviewer(s) as well.