Skip to content

337 sudo new sections#433

Open
Nolab0 wants to merge 9 commits intomainfrom
337-sudo-new-sections
Open

337 sudo new sections#433
Nolab0 wants to merge 9 commits intomainfrom
337-sudo-new-sections

Conversation

@Nolab0
Copy link
Member

@Nolab0 Nolab0 commented Feb 24, 2026

Close #337

@Nolab0 Nolab0 requested a review from supun-io February 24, 2026 15:49
Copy link
Member

@supun-io supun-io left a comment

Choose a reason for hiding this comment

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

Copy link
Member

@supun-io supun-io left a comment

Choose a reason for hiding this comment

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

UI Changes

  • Newsletters
    • Set block=false for text input in search input
    • Add TextInput to filter by organization ID
    • Show organization details in each row
    • Sorting
      • Remove Name sort (not useful)
      • ID ASC
      • ID DESC (default)
      • Most issues
    • Dump the newsletter object
    • Use a UI for stats (similar to the one we have in the Console)+
  • Issues
    • Filter by newsletter at the top with a dropdown to select the newsletter (can be set via query param). Add a link in /newsletters/{id} to see its issues here with link
    • dump the issue object, not the error (private)
    • link back to the newsletter seems to be broken

@@ -0,0 +1 @@
export const ssr = false;
Copy link
Member

Choose a reason for hiding this comment

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

you don't need ssr here when added to a parent layout

Copy link
Member

Choose a reason for hiding this comment

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

remove file


<h3>Stats</h3>

<CodeBlock code={JSON.stringify(stats, null, 2)} language="json" />
Copy link
Member

Choose a reason for hiding this comment

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

ah, I asked to dump the newsletter object.

It would be better to use a UI for stats

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.

[sudo] new sections

2 participants