[D8] Kaltura Video Embed Field

Added PAReview checklist link.

Hi @Abhinand Gokhala K,

Thank you for contribution!!!

Hi @Abhinand Gokhala K,

From the below function, I can see that you're setting partner_id, uiconf_id in the configuration and also providing the config form to change these values. Is this are the dummy or trail data or the placeholder for the textfield.
* If trail, what happens when the trial ends.
* If placeholder then directly add placeholder properties to textfield.

I'll recommend not save in the configuration while installing instead provide these details in the documentation.

function video_embed_kaltura_install() {

  // Kaltura default configuration.
  $config = \Drupal::configFactory()->getEditable('video_embed_kaltura.kaltura_config');
  $config->set('partner_id', 811441);
  $config->set('uiconf_id', 32783592);
  $config->save();
}

kaltura settings:
  title: 'Kaltura video settings'

The permissions should start with a verb, as in change own username or administer user settings.

Thank you for the contribution!

I have not found any security related issue into the code.
I set the issue status to Reviewed & tested by the community.

Thank you for your contribution! I am going to update your account.

These 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 more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, 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.

I thank all the dedicated reviewers as well.