Skip to content

Migrate explore euid flyouts thi#6

Merged
YulNaumenko merged 4 commits intoYulNaumenko:migrate-explore-euid-pr1-flyoutsfrom
PhilippeOberti:migrate-explore-euid-flyouts-thi
Mar 26, 2026
Merged

Migrate explore euid flyouts thi#6
YulNaumenko merged 4 commits intoYulNaumenko:migrate-explore-euid-pr1-flyoutsfrom
PhilippeOberti:migrate-explore-euid-flyouts-thi

Conversation

@PhilippeOberti
Copy link
Copy Markdown

Summary

This PR fixes a couple of small things on the main PR:

  • remove unused code
  • remove unwanted re-added coded during a recent rebase/merge
  • removed some constants in favor of existing code
  • extracted some logic into shared functions for less code duplication

I ran a few commands locally to make sure things were not introducing any issues:

Typescheck is ✅
Screenshot 2026-03-26 at 4 21 38 PM

Flyout folder tests are ✅
Screenshot 2026-03-26 at 4 26 39 PM

Flyout_v2 folder tests are ✅
Screenshot 2026-03-26 at 4 27 17 PM

@YulNaumenko YulNaumenko merged commit 071f614 into YulNaumenko:migrate-explore-euid-pr1-flyouts Mar 26, 2026
8 checks passed
@PhilippeOberti PhilippeOberti deleted the migrate-explore-euid-flyouts-thi branch March 26, 2026 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants