Skip to content

Conversation

@kenny-not-dead
Copy link
Contributor

close #5972

When trying to fix the bug, given that we had previously used a check for the presence of an error message in this cell by key, I wondered if this was necessary. So, two different errors would generally be shown in one cell, and given that we had already encountered two errors with the same text, it seemed appropriate to me.

In fact, I think that this problem, or rather the clearing of other error messages, still requires attention, perhaps they should be cleared when a new one appears... but this is a subject for another discussion

Before:

described in issue

After:
image

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).

@univer-bot univer-bot bot added the qa:untested This PR is ready to be tested label Oct 27, 2025
@github-actions
Copy link

github-actions bot commented Oct 27, 2025

View Deployment

📑 Demo (React@19) Demo (React@16) 📚 Storybook
🔗 Preview link 🔗 Preview link 🔗 Preview link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

qa:untested This PR is ready to be tested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] 2 errors with the same text

1 participant