fix: Validation for cloud name

[HeetVekariya]

Description

Closes #58

• Add validation for the environment variable VITE_PUBLIC_CLOUDINARY_CLOUD_NAME

Changes made

• Added a custom function checkCloudinaryCloudName in the cloudinary.ts, which accepts the cloud name as a parameter and throws error if it is null

• Imported that function in the below files and passed environment variable as a parameter:

  • CldVideoPlayer.svelte (svelte-cloudinary/src/lib/components/CldVideoPlayer.svelte)
  • CldUploadWidget.svelte (svelte-cloudinary/src/lib/components/CldUploadWidget.svelte)

• Note : When i run pnpm install it gives me error, so i am not able to test it locally

Issue Ticket Number

Fixes #58

Type of change

• [ ] Bug fix (non-breaking change which fixes an issue)
• [x] New feature (non-breaking change which adds functionality)
• [x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
• [ ] Fix or improve the documentation
• [ ] This change requires a documentation update

Checklist

• [x] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
• [ ] I have created an issue ticket for this PR
• [x] I have checked to ensure there aren't other open Pull Requests for the same update/change?
• [x] I have performed a self-review of my own code
• [ ] I have run tests locally to ensure they all pass
• [ ] I have commented my code, particularly in hard-to-understand areas
• [ ] I have made corresponding changes needed to the documentation

[vercel[bot]]

@HeetVekariya is attempting to deploy a commit to the Cloudinary DevX Team on Vercel.

A member of the Team first needs to authorize it.

[colbyfayock]

hey @matiasfha we had some chatter in the Space Jelly discord in the #development channel if you want to get a bit of context here, have some thoughts id love to get your input on

[vercel[bot]]

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

Name	Status	Preview	Comments	Updated (UTC)
svelte-cloudinary	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 12, 2023 7:15pm

[matiasfha]

Looks great!. I liked the approach. the CldImage and CldOgImage component also requires this validation. This can be added into the component itself or here

[HeetVekariya]

@matiasfha Thank you for your comment, as suggested the changes are implemented.

[HeetVekariya]

Also please double check my imports, may it can be wrong

[matiasfha]

Works great! Thanks for your work and the PR

[matiasfha]

@all-contributors please add @HeetVekariya for code

[allcontributors[bot]]

@matiasfha

I've put up a pull request to add @HeetVekariya! :tada:

[github-actions[bot]]

:tada: This PR is included in version 1.1.1 :tada:

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot :package::rocket:

[colbyfayock]

@HeetVekariya congrats on the merged PR! since it's officially October, you've always qualified for Hacktoberfest swag! please email hacktoberfest@cloudinary.com with your github username and a link to your contribution where I'll follow up to request more information for getting you your swag

https://cloudinary.com/blog/hacktoberfest-celebrate-open-source-sdks