Skip to content

refactor: resource_team_id_query#3479

Merged
Ziinc merged 6 commits into
Logflare:mainfrom
msmithstubbs:refactor/resouce-team-id
Jun 8, 2026
Merged

refactor: resource_team_id_query#3479
Ziinc merged 6 commits into
Logflare:mainfrom
msmithstubbs:refactor/resouce-team-id

Conversation

@msmithstubbs

@msmithstubbs msmithstubbs commented May 15, 2026

Copy link
Copy Markdown
Contributor

Small refactor followup on default team param PRs.

  • reduce repetition in TeamContext
  • consistency in router

Ready to merge after:

@msmithstubbs msmithstubbs force-pushed the refactor/resouce-team-id branch 3 times, most recently from a8760aa to 2edb782 Compare May 20, 2026 00:39
@msmithstubbs msmithstubbs force-pushed the refactor/resouce-team-id branch from 2edb782 to 09eb20c Compare May 25, 2026 23:36
@msmithstubbs msmithstubbs marked this pull request as ready for review May 26, 2026 00:00
@msmithstubbs msmithstubbs force-pushed the refactor/resouce-team-id branch 3 times, most recently from 9eff0a9 to 46f39b2 Compare June 1, 2026 02:03
@msmithstubbs msmithstubbs force-pushed the refactor/resouce-team-id branch 3 times, most recently from 45a1c10 to 7ee78f1 Compare June 4, 2026 23:47
@msmithstubbs msmithstubbs force-pushed the refactor/resouce-team-id branch from 7ee78f1 to 844e5c3 Compare June 8, 2026 00:15
html = submit_query_form(view, conn)

assert html =~ "Ran query successfully"
assert_patch(view) =~ ~s(t=#{other_team.id})

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

we should add in a small flash each time we switch teams as well. i think now flashes will stack on top of each other, would be good to have a more robust toast system.

@Ziinc Ziinc merged commit f159b26 into Logflare:main Jun 8, 2026
13 checks passed
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