taro icon indicating copy to clipboard operation
taro copied to clipboard

fix: 修复H5路由有 basename和customRoutes 的情况下navigate 报错

Open sliwei opened this issue 10 months ago • 4 comments

这个 PR 做了什么? (简要描述所做更改)

修复H5路由有 basename和customRoutes 的情况下navigate 报错

这个 PR 是什么类型? (至少选择一个)

  • [x] 错误修复(Bugfix) issue: fix #
  • [ ] 新功能(Feature)
  • [ ] 代码重构(Refactor)
  • [ ] TypeScript 类型定义修改(Typings)
  • [ ] 文档修改(Docs)
  • [ ] 代码风格更新(Code style update)
  • [ ] 其他,请描述(Other, please describe):

这个 PR 涉及以下平台:

  • [ ] 所有小程序
  • [ ] 微信小程序
  • [ ] 支付宝小程序
  • [ ] 百度小程序
  • [ ] 字节跳动小程序
  • [ ] QQ 轻应用
  • [ ] 京东小程序
  • [ ] 快应用平台(QuickApp)
  • [x] Web 平台(H5)
  • [ ] 移动端(React-Native)
  • [ ] 鸿蒙(harmony)

sliwei avatar Apr 14 '24 14:04 sliwei

欢迎提交 PR~ Taro 非常感谢您对开源事业做出的贡献!🌷🌷🌷

一般 PR 会在一到两周内进行 review,成功合入后会随下一个版本进行发布。

Review 需要耗费大量时间,所以请遵循以下规范,协助我们提高 review 效率🙏🙏🙏

  1. 详细介绍 PR 的背景(非常重要,例如解决了什么问题,该问题如何复现等)
  2. 确保 CI 顺利运行。
  3. 最好能提供对应的测试用例。

为了更好地进行沟通,请加入 Taro 开发者微信群:

taro-bot2[bot] avatar Apr 14 '24 14:04 taro-bot2[bot]

欢迎提交 PR~ Taro 非常感谢您对开源事业做出的贡献!🌷🌷🌷

一般 PR 会在一到两周内进行 review,成功合入后会随下一个版本进行发布。

Review 需要耗费大量时间,所以请遵循以下规范,协助我们提高 review 效率🙏🙏🙏

  1. 详细介绍 PR 的背景(非常重要,例如解决了什么问题,该问题如何复现等)
  2. 确保 CI 顺利运行。
  3. 最好能提供对应的测试用例。

为了更好地进行沟通,请加入 Taro 开发者微信群:

taro-bot2[bot] avatar Apr 14 '24 14:04 taro-bot2[bot]

Codecov Report

Attention: Patch coverage is 0% with 10 lines in your changes are missing coverage. Please review.

Project coverage is 58.62%. Comparing base (02b845d) to head (9d42da0).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #15508      +/-   ##
==========================================
- Coverage   58.64%   58.62%   -0.03%     
==========================================
  Files         500      500              
  Lines       21936    21946      +10     
  Branches     5456     5600     +144     
==========================================
  Hits        12865    12865              
- Misses       8052     8064      +12     
+ Partials     1019     1017       -2     
Flag Coverage Δ
taro-cli 75.00% <ø> (ø)
taro-runner 54.00% <ø> (ø)
taro-runtime 79.39% <ø> (ø)
taro-web 44.64% <0.00%> (-0.07%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
packages/taro-router/src/router/spa.ts 9.81% <0.00%> (-0.65%) :arrow_down:

... and 2 files with indirect coverage changes

codecov[bot] avatar May 06 '24 06:05 codecov[bot]

您好,taro-router 在是否为当前页面的逻辑判断上有点混乱,我这边统一梳理了一下,https://github.com/NervJS/taro/pull/15658, 您这个pr要修复的问题应该被包括在里面了,我先关闭一下,十分感谢您的贡献,有问题我们可以继续沟通~

ZEJIA-LIU avatar May 07 '24 06:05 ZEJIA-LIU