Skip to content

Conversation

@jordienr
Copy link
Member

@jordienr jordienr commented Aug 13, 2025

removes redirect effect to sned user to the latest visited logs page,

since the unified logs effect overwrites it, i think its causing issues

@jordienr jordienr requested a review from a team as a code owner August 13, 2025 09:47
@supabase
Copy link

supabase bot commented Aug 13, 2025

This pull request has been ignored for the connected project xguihxuzqibwxjnimxev because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@vercel
Copy link

vercel bot commented Aug 13, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Project Deployment Preview Comments Updated (UTC)
studio-self-hosted Ready Preview Comment Aug 13, 2025 0:47am
studio-staging Ready Preview Comment Aug 13, 2025 0:47am
6 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
cms ⬜️ Ignored Aug 13, 2025 0:47am
studio ⬜️ Ignored Aug 13, 2025 0:47am
design-system ⬜️ Skipped Aug 13, 2025 0:47am
docs ⬜️ Skipped Aug 13, 2025 0:47am
ui-library ⬜️ Skipped Aug 13, 2025 0:47am
zone-www-dot-com ⬜️ Skipped Aug 13, 2025 0:47am

@coveralls
Copy link

coveralls commented Aug 13, 2025

Coverage Status

coverage: 71.086%. remained the same
when pulling 56c3061 on jordi/rm-last-visited-logs-effect
into d60b3e3 on master.

Copy link
Member

@joshenlim joshenlim left a comment

Choose a reason for hiding this comment

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

LGTM! Verified on staging preview 🙂

@jordienr jordienr merged commit c03dcc1 into master Aug 13, 2025
19 of 20 checks passed
@jordienr jordienr deleted the jordi/rm-last-visited-logs-effect branch August 13, 2025 13:37
PLyn pushed a commit that referenced this pull request Aug 25, 2025
* rm effect

* diff approach, rm effect for unified logs

* simplify, only redirect to explorer, keep unifiedlog check

* clean up

* dynamic url in the sidebar based on unified logs feature preview

* fix deep link missing ref
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.

3 participants