Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix bug after porting tab manager #1330

Closed
wants to merge 4 commits into from
Closed

Fix bug after porting tab manager #1330

wants to merge 4 commits into from

Conversation

JYC333
Copy link
Contributor

@JYC333 JYC333 commented Mar 3, 2025

  • Revert "Revert "port tab manager to ts""
  • fix: 🐛 fix wrong triggerEvent usage

It's related to our event handling system, I didn't learn it right before. But now I think I should understand it correctly. I create a task for refactoring the event handling system since I think it's quite hard to maintance now. #1331

Copy link
Contributor

@eliandoran eliandoran left a comment

Choose a reason for hiding this comment

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

It appears some of the end-to-end tests are failing, whereas they are not failing on the latest develop. Could you please check?

@eliandoran eliandoran marked this pull request as draft March 5, 2025 11:38
@JYC333 JYC333 marked this pull request as ready for review March 5, 2025 12:44
@JYC333 JYC333 marked this pull request as draft March 5, 2025 12:44
@JYC333 JYC333 closed this Mar 5, 2025
@JYC333 JYC333 deleted the fix branch March 18, 2025 13:23
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants