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.

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

增强文件校验的兼容性 #1929

Closed
wants to merge 1 commit into from
Closed

Conversation

Azir-11
Copy link
Contributor

@Azir-11 Azir-11 commented Nov 8, 2024

  1. 能找到的地方都加了个toLowerCase
  2. 优化了一点点image.js的函数写法

Copy link
Collaborator

@pixelmaxQm pixelmaxQm left a comment

Choose a reason for hiding this comment

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

有一处需要确认

@@ -212,7 +212,7 @@ const listObj = {
const chooseImg = (url) => {
if (props.fileType) {
const typeSuccess = listObj[props.fileType].some(item => {
if (url.includes(item)) {
if (url.includes(item.toLowerCase())) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

这里写反了吧,应该是对url进行toLowerCase

@Azir-11 Azir-11 closed this Nov 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants