Merge pull request #1586 from DonKirkham/upgrade-react-carousel

samples/react-carousel/.yo-rc.json

@@ -1,6 +1,6 @@
 {
   "@microsoft/generator-sharepoint": {
-    "version": "1.10.0",
+    "version": "1.11.0",
     "libraryName": "react-carousel",
     "libraryId": "263a1d21-f4c7-4a46-a364-e5f995286b9b",
     "environment": "spo",

samples/react-carousel/README.md

@@ -27,7 +27,7 @@ It uses Microsoft Graph API to get image/video url and use PnPjs to load files f
 
 ## Used SharePoint Framework Version
 
-![1.10](https://img.shields.io/badge/version-1.10.0-green.svg)
+![1.11](https://img.shields.io/badge/version-1.11.0-green.svg)
 
 ## Applies to
 
@@ -60,6 +60,7 @@ Solution|Author(s)
 --------|---------
 Carousel Image/Video  Web Part|João Mendes
 Carousel Image/Video  Web Part|Rahul Suryawanshi ([@rahulsuryawansh](https://twitter.com/rahulsuryawansh))
+Carousel Image/Video  Web Part|Don Kirkham ([@DonKirkham](https://twitter.com/DonKirkham))
 
 ## Version history
 
@@ -67,6 +68,7 @@ Version|Date|Comments
 -------|----|--------
 1.0.0|July 11, 2019|Initial release
 2.0.0|June 17, 2020|Upgraded to SPFx v1.10.0 (Rahul Suryawanshi)
+3.0.0|October 31, 2020|Upgraded to SPFx v1.11.0 (Don Kirkham)
 
 
 ## Disclaimer

samples/react-carousel/config/package-solution.json

@@ -3,10 +3,17 @@
 	"solution": {
 		"name": "react-carousel-client-side-solution",
 		"id": "263a1d21-f4c7-4a46-a364-e5f995286b9b",
-    "version": "2.0.0.0",
+		"version": "3.0.0.0",
 		"includeClientSideAssets": true,
 		"skipFeatureDeployment": true,
-    "isDomainIsolated": false
+		"isDomainIsolated": false,
+		"developer": {
+			"name": "Contoso",
+			"privacyUrl": "https://contoso.com/privacy",
+			"termsOfUseUrl": "https://contoso.com/terms-of-use",
+			"websiteUrl": "https://contoso.com/my-app",
+			"mpnId": "000000"
+		}
 	},
 	"paths": {
 		"zippedPackage": "solution/react-carousel.sppkg"

samples/react-carousel/package.json

@@ -14,12 +14,12 @@
     "test:watch": "./node_modules/.bin/jest --config ./config/jest.config.json --watchAll"
   },
   "dependencies": {
-    "@microsoft/sp-core-library": "1.10.0",
-    "@microsoft/sp-http": "1.10.0",
-    "@microsoft/sp-lodash-subset": "1.10.0",
-    "@microsoft/sp-office-ui-fabric-core": "1.10.0",
-    "@microsoft/sp-property-pane": "1.10.0",
-    "@microsoft/sp-webpart-base": "1.10.0",
+    "@microsoft/sp-core-library": "1.11.0",
+    "@microsoft/sp-http": "1.11.0",
+    "@microsoft/sp-lodash-subset": "1.11.0",
+    "@microsoft/sp-office-ui-fabric-core": "1.11.0",
+    "@microsoft/sp-property-pane": "1.11.0",
+    "@microsoft/sp-webpart-base": "1.11.0",
     "@microsoft/teams-js": "^1.6.0",
     "@pnp/common": "^1.3.11",
     "@pnp/graph": "^1.3.11",
@@ -46,10 +46,10 @@
   },
   "devDependencies": {
     "@microsoft/rush-stack-compiler-3.3": "0.3.5",
-    "@microsoft/sp-build-web": "1.10.0",
-    "@microsoft/sp-module-interfaces": "1.10.0",
-    "@microsoft/sp-tslint-rules": "1.10.0",
-    "@microsoft/sp-webpart-workbench": "1.10.0",
+    "@microsoft/sp-build-web": "1.11.0",
+    "@microsoft/sp-module-interfaces": "1.11.0",
+    "@microsoft/sp-tslint-rules": "1.11.0",
+    "@microsoft/sp-webpart-workbench": "1.11.0",
     "@types/chai": "3.4.34",
     "@types/mocha": "2.2.38",
     "@types/react": "16.8.8",

samples/react-carousel/src/webparts/carousel/components/Carousel.tsx

@@ -85,10 +85,14 @@ export default class Carousel extends React.Component<ICarouselProps, ICarouselS
 
 				let mediaType: string = '';
 				switch (image.File_x0020_Type) {
-          case ('jpg' || 'jpeg' || 'png' || 'tiff' || 'gif'):
+					case 'jpg':
+					case 'jpeg':
+					case 'png':
+					case 'tiff':
+					case 'gif':
 						mediaType = 'image';
 						break;
-          case ('mp4'):
+					case 'mp4':
 						mediaType = 'video';
 						break;
 					default:

samples/react-carousel/tsconfig.json

@@ -29,7 +29,8 @@
     ]
   },
   "include": [
-    "src/**/*.ts"
+	"src/**/*.ts",
+	"src/**/*.tsx"
   ],
   "exclude": [
     "node_modules",