photopicker component imports CognitoIdentify client unnecessarily

[srchase]

Hello,

An issue has been opened on the JavaScript SDK: aws/aws-sdk-js#2192

Specifically, the photopicker component is causing issues when importing the CognitoIdentity client.

amplify-js/packages/aws-amplify-angular/src/components/storage/photo-picker-component/photo-picker.factory.ts

Line 8 in 8f6c448

import { String } from 'aws-sdk/clients/cognitoidentity';

I don't believe that component needs to import the client.

I'm happy to submit a PR if it would be helpful to correct this issue.

[AllanZhengYP]

Basically in aws-sdk v2.287.0, the unused typings are removed from the model, which contains the String type referred here. You can just remove the String import and instead use normal string types here.

[haverchuck]

The { String } import has been removed from photo-picker-factory.ts.

[ShaneCotter]

I am still getting this error even though in my photo-picker-factory.ts there is no String import on line 3. Can someone help please.

[appwerkz]

Yes, I am getting the same error too.

ERROR in node_modules/aws-amplify-angular/lib/components/storage/photo-picker-component/photo-picker.factory.d.ts(3,10): error TS2305:
Module '"{hard coded path}/aws-activities/aws-amplify-ionic/aws-amplify-ionic-sample/node_modules/aws-sdk/clients/cognitoidentity"' has no exported member 'String'.

[harsh594]

I replaced import { String } from 'aws-sdk/clients/cognitoidentity'; with import * as String from 'aws-sdk/clients/cognitoidentity'; and it works for me now.

[github-actions]

This issue has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs.

Looking for a help forum? We recommend joining the Amplify Community Discord server *-help channels or Discussions for those types of questions.