-
Notifications
You must be signed in to change notification settings - Fork 257
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
docs: add docs for StorageImage #4259
Conversation
|
At a minimum you must include the `alt`, `imgKey` and `accessLevel` props. `accessLevel` refers to the [Amplify Storage access level](https://docs.amplify.aws/lib/storage/configureaccess/q/platform/js/), which is `'public' | 'private' | 'protected'`. | ||
|
||
<Example> | ||
<Image alt='cat' src='/cats/1.jpg' width="400px" height="400px" /> |
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.
The existing storage mock is doing via a custom provider and it is not supported on StorageImage
API. So simply use an Image
to render the example.
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.
I try to deploy an app to hosting but now it requires admin access to our repo 😅
@zchenwei I deployed it here https://chenwz-dev-imagedocs.dvmvffzts1tcu.amplifyapp.com/ (probably still building if you are reading this just after I posted this comment though) |
Ohhh sweet! I ended up successfully deploying an app with hosting a few minutes ago. It is extremely hard to debug in hosting console with long logs 😭 <- https://chenwz-dev-imagedocs.daoba3qczgnpf.amplifyapp.com/react/connected-components/storage/storageimage |
@zchenwei missing snapshot update for sitemap :) (my build failed on that) |
docs/src/pages/[platform]/connected-components/storage/storageimage/react.mdx
Outdated
Show resolved
Hide resolved
docs/src/pages/[platform]/connected-components/storage/storageimage/props.ts
Show resolved
Hide resolved
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.
LGTM
Description of changes
Add documentation for
StorageImage
Issue #, if available
Description of how you validated changes
Checklist
yarn test
passes and tests are updated/addedsideEffects
field updatedBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.