Skip to content
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.

Sign up for GitHub

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

fix(rn): 修复previewImage 预览图片API的current必须是urls其中的一个 #11632

Open
wants to merge 8 commits into
base: 3.x
Choose a base branch
from
Open

fix(rn): 修复previewImage 预览图片API的current必须是urls其中的一个 #11632

Changes from 2 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
59639fe
fix: 修复previewImage 预览图片API的current必须是urls其中的一个
Apr 13, 2022
34088cc
Merge branch 'next' into feature-fix-previewImage
zhiqingchen Apr 13, 2022
c00c1e7
fix: 优化previewImage预览图片api的传参校验逻辑
Apr 19, 2022
d34770c
Merge branch 'next' into feature-fix-previewImage
zhiqingchen Apr 19, 2022
50d7845
fix: 优化previewImage预览图片api的传参校验逻辑
Apr 20, 2022
c05caab
Merge branch 'feature-fix-previewImage' of https://github.com/WhiteSe…
Apr 20, 2022
7e872cb
fix: 优化预览图片校验逻辑
Apr 21, 2022
c668232
Merge branch 'next' into feature-fix-previewImage
zhiqingchen Apr 22, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions packages/taro-rn/src/lib/previewImage/index.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,9 @@ export function previewImage(obj: Taro.previewImage.Option): void {
complete
}: any = obj || {}
const index = urls.indexOf(current)
if (index === -1) {
throw new Error('"current" or "urls" is invalid')
}
// if (index === -1) {
// throw new Error('"current" or "urls" is invalid')
// }

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修改后把 commit 合并成一个就行了

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个我们 squash and merge 就会是一个commit

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

哦哦 我以为要先合并呢😂

let sibling
function backhandler() {
Expand Down