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
Updated TeamsJS CC Matrix App #2074
base: main
Are you sure you want to change the base?
Conversation
apps/teamsjs-cc-app/readme.md
Outdated
@@ -0,0 +1,93 @@ | |||
# TeamsJS Capability Checker |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this file have some info about what this app is and what it's for?
Some of the info about how to run it is good and helpful and should stay -- but some details specific to this app would be good.
apps/teamsjs-cc-app/tsconfig.json
Outdated
"resolveJsonModule": true, | ||
"isolatedModules": true, | ||
"noEmit": true, | ||
"jsx": "react-jsx" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider alphabetizing these
"manifestVersion": "1.16", | ||
"version": "1.0.2", | ||
"id": "${{TEAMS_APP_ID}}", | ||
"packageName": "com.microsoft.teams.extension", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For future reference, putting screenshots of the app into the PR request can be helpful to reviewers. |
I'll review more of the code once the tests are passing and the code has been updated to latest main. |
For more information about how to contribute to this repo, visit this page.
Description
Main changes in the PR:
Validation
Validation performed:
None
Unit Tests added:
Not necessary, sample app
End-to-end tests added:
No
Additional Requirements
Change file added:
Not needed as this doesn't impact client facing code