Skip to content

feat: allow published data to be unauth#2314

Open
minottic wants to merge 2 commits intomasterfrom
public_published
Open

feat: allow published data to be unauth#2314
minottic wants to merge 2 commits intomasterfrom
public_published

Conversation

@minottic
Copy link
Copy Markdown
Member

@minottic minottic commented Apr 8, 2026

Description

This makes published data overview and {id} public,
since the endpoint allow non-auth access restricting to registered states.
It hides the buttons when non auth to avoid confusion

Summary by Sourcery

Make published data overview and details pages publicly accessible while gating modification actions behind login status.

New Features:

  • Allow access to the published data dashboard and individual published data details routes without authentication.

Enhancements:

  • Hide published data status/action buttons on the details page when the user is not logged in, showing them only for authenticated users.
  • Track login state in the published data details component via the store and update tests to cover login-dependent button visibility and store-driven state.

minottic added 2 commits April 8, 2026 14:37
This makes published data overview and {id} public,
since the endpoint allow non-auth access restricting to registered states.
It hides the buttons when non auth to avoid confusion
@minottic minottic marked this pull request as ready for review April 8, 2026 14:46
@minottic minottic requested a review from a team as a code owner April 8, 2026 14:46
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

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.

1 participant