The plugin-meetings previously had weak tsconfig rules. The purpose of this PR is to improve the tsconfig rules to help identify issues faster with the data across this plugin. I am looking forward to working together with you to enhance the SDK code.
Any suggestions are welcome.
In this PR, I do not intend to fix all the types, but rather enable better handling of types.
by making the following changes
I would like to explain some of my refactoring ideas here:
I have used the correct types wherever possible, and I have used unknown for types that I am not certain about. In cases where there is an object, you may see Record<string, any>, and there were a couple of instances with any that were difficult to fix easily because they required changes in the codebase.
Additionally, I have added a global.d.ts file to mock types from other plugins. The idea is to assist TypeScript in working correctly for now, and once these plugins are changed, these declarations should be deleted along with the file.
Change Type
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
COMPLETES WEBEX-390013
This pull request addresses
The
plugin-meetings
previously had weaktsconfig
rules. The purpose of this PR is to improve thetsconfig
rules to help identify issues faster with the data across this plugin. I am looking forward to working together with you to enhance the SDK code.Any suggestions are welcome.
In this PR, I do not intend to fix all the types, but rather enable better handling of types.
by making the following changes
I would like to explain some of my refactoring ideas here:
I have used the correct types wherever possible, and I have used
unknown
for types that I am not certain about. In cases where there is an object, you may seeRecord<string, any>
, and there were a couple of instances withany
that were difficult to fix easily because they required changes in the codebase.Additionally, I have added a
global.d.ts
file to mock types from other plugins. The idea is to assist TypeScript in working correctly for now, and once these plugins are changed, these declarations should be deleted along with the file.Change Type
The following scenarios where tested
I certified that
[x] I have read and followed contributing guidelines
[x] I discussed changes with code owners prior to submitting this pull request
[x] I have not skipped any automated checks
[x] All existing and new tests passed
[x] I have updated the documentation accordingly
Make sure to have followed the contributing guidelines before submitting.