fix(sheet): sheet name normalize string

Dushusir commented 5 days ago

close #4076 close #4002 close #3902 (Not reproduced, just test it)

Pull Request Checklist

[ ] Related tickets or issues have been linked in the PR description (or missing issue).
[ ] Naming convention is followed (do please check it especially when you created new plugins, commands and resources).
[ ] Unit tests have been added for the changes (if applicable).
[ ] Breaking changes have been documented (or no breaking changes introduced in this PR).

github-actions[bot] commented 5 days ago

View Deployment

📑 Examples	📚 Storybook
🔗 Preview link	🔗 Preview link

github-actions[bot] commented 5 days ago

Playwright test results

passed 20 passed

Details

stats 20 tests across 9 suites
4 minutes, 48 seconds
commit 080a270
info For more information, [see full report]()

codecov[bot] commented 5 days ago

Codecov Report

Attention: Patch coverage is 68.42105% with 6 lines in your changes missing coverage. Please review.

Project coverage is 33.79%. Comparing base (59b95fa) to head (080a270). Report is 2 commits behind head on dev.

Files with missing lines	Patch %	Lines
...src/controllers/formula-alert-render.controller.ts	0.00%	6 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## dev #4172 +/- ## ======================================= Coverage 33.78% 33.79% ======================================= Files 2485 2485 Lines 127547 127561 +14 Branches 28594 28594 ======================================= + Hits 43093 43104 +11 - Misses 84454 84457 +3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

dream-num / univer