Skip to content

Show count of puzzles in each table #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Show count of puzzles in each table #46

wants to merge 2 commits into from

Conversation

julioalucero
Copy link
Contributor

@julioalucero julioalucero commented Oct 16, 2025 •
edited
Loading

Feature requested at #43.

Screenshot 2025-10-17 at 10 51 37 am

@julioalucero julioalucero self-assigned this Oct 16, 2025
FionaDL
FionaDL approved these changes Oct 17, 2025
View reviewed changes
Copy link
Member

@FionaDL FionaDL left a comment

Choose a reason for hiding this comment

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

This looks good to me!

@julioalucero
Copy link
Contributor Author

@FionaDL I did a few improvements after pinging Ariel.

  1. I removed the counter under Rejected Puzzles
  2. Use length instead of count (it loads all the records into memory first and then does a headcount there)
  3. And removed the style, which was not necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@FionaDL FionaDL FionaDL approved these changes

Assignees

@julioalucero julioalucero

Labels

Ready for QA

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@julioalucero @FionaDL