针对关于允许修改解锁音乐信息的建议(issue #216)的解决方案 #218

Open

xhacker-zzz wants to merge 0 commits from unknown repository into master

pull from: unknown repository

merge into: um:master

um:master

um:gh-pages-redirect

um:gh-pages

Conversation 10 Commits - Files Changed 89 +32417 -5026

xhacker-zzz commented 2021-12-22 08:02:05 +00:00 (Migrated from github.com)

Copy Link

No description provided.

ix64 (Migrated from github.com) reviewed 2021-12-22 08:02:05 +00:00

ix64 commented 2021-12-22 08:12:28 +00:00 (Migrated from github.com)

Copy Link

似乎并没有重写meta的逻辑？

似乎并没有重写meta的逻辑？

xhacker-zzz commented 2021-12-22 08:16:53 +00:00 (Migrated from github.com)

Copy Link

似乎并没有重写meta的逻辑？

scope.row.xxx=xxx改不了meta吗？我不太确定

> 似乎并没有重写meta的逻辑？ scope.row.xxx=xxx改不了meta吗？我不太确定

ix64 commented 2021-12-22 08:21:27 +00:00 (Migrated from github.com)

Copy Link

scope.row.xxx=xxx改不了meta吗？我不太确定

需要调用utils中 WriteMetaToMp3 WriteMetaToFlac 写入并重新创建 blob 才行
否则只是修改了网页上显示的信息，而非文件的meta

> scope.row.xxx=xxx改不了meta吗？我不太确定 需要调用utils中 `WriteMetaToMp3` `WriteMetaToFlac` 写入并重新创建 blob 才行 否则只是修改了网页上显示的信息，而非文件的meta

xhacker-zzz commented 2021-12-22 14:31:51 +00:00 (Migrated from github.com)

Copy Link

@ix64 现在应该可以了。你测试一下？

@ix64 现在应该可以了。你测试一下？

ix64 (Migrated from github.com) requested changes 2021-12-22 14:37:15 +00:00

ix64 commented 2021-12-22 14:44:01 +00:00 (Migrated from github.com)

Copy Link

代码仍有问题
在提交之前，可以在本地运行：
npm run build 进行构建测试
npm run pretty 对源代码进行格式化
npm run test 进行单元测试

代码仍有问题 在提交之前，可以在本地运行： `npm run build` 进行构建测试 `npm run pretty` 对源代码进行格式化 `npm run test` 进行单元测试

ix64 commented 2021-12-22 14:49:24 +00:00 (Migrated from github.com)

Copy Link

我没环境啊😅

GitHub 有提供在线 IDE

> 我没环境啊😅  GitHub 有提供在线 IDE

xhacker-zzz commented 2021-12-22 17:54:47 +00:00 (Migrated from github.com)

Copy Link

代码仍有问题 在提交之前，可以在本地运行： npm run build 进行构建测试 npm run pretty 对源代码进行格式化 npm run test 进行单元测试

现在代码应该没问题了，但是这

是啥情况？
另外，网页版的IDE好像不能安装npn

> 代码仍有问题 在提交之前，可以在本地运行： `npm run build` 进行构建测试 `npm run pretty` 对源代码进行格式化 `npm run test` 进行单元测试 现在代码应该没问题了，但是这  是啥情况？ 另外，网页版的IDE好像不能安装npn

ix64 commented 2021-12-22 19:13:31 +00:00 (Migrated from github.com)

Copy Link

另外，网页版的IDE好像不能安装npm

Web CLI 自带了npm的

CI 是一个上游的 bug 导致的，暂时不用理会

可能会迟一点审阅合并，暂时没空

> 另外，网页版的IDE好像不能安装npm Web CLI 自带了npm的 CI 是一个上游的 bug 导致的，暂时不用理会 可能会迟一点审阅合并，暂时没空

ix64 commented 2021-12-22 19:16:24 +00:00 (Migrated from github.com)

Copy Link

请你实际运行过所改动的代码，再发送审阅请求

 请你实际运行过所改动的代码，再发送审阅请求

ix64 commented 2021-12-22 19:22:49 +00:00 (Migrated from github.com)

Copy Link

还有几点：

• 编辑界面不应该默认显示（大部分用户没有编辑的需求）
• 将 meta 写入文件不应该 在 onchange 执行，此操作会消耗大量CPU，是一个耗时的操作，应该由用户主动点击触发

这个 feature request，idea是很棒的，但请尝试解决以上问题，否则体验会很糟糕；
如果无法解决，可以暂时 将该 pr 转为 draft，或 close 掉此 pr 等待我们处理

还有几点： - 编辑界面不应该默认显示（大部分用户没有编辑的需求） - 将 meta 写入文件不应该 在 `onchange` 执行，此操作会消耗大量CPU，是一个耗时的操作，应该由用户主动点击触发 这个 feature request，idea是很棒的，但请尝试解决以上问题，否则体验会很糟糕； 如果无法解决，可以暂时 将该 pr 转为 draft，或 close 掉此 pr 等待我们处理

This repo is archived. You cannot comment on pull requests.

Reviewers

No reviewers

ix64

Labels

Clear labels   

bug

Something isn't working

  

dependencies

Pull requests that update a dependency file

  

discuss

  

documentation

Improvements or additions to documentation

  

duplicate

This issue or pull request already exists

  

enhancement

New feature or request

  

good first issue

Good for newcomers

  

help wanted

Extra attention is needed

  

invalid

This doesn't seem right

  

question

Further information is requested

  

timeout

Response timeout

  

wontfix

This will not be worked on

No Label

bug

dependencies

discuss

documentation

duplicate

enhancement

good first issue

help wanted

invalid

question

timeout

wontfix

Milestone

Clear milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

No Assignees

1 Participants

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: um/web-archived#218

No description provided.