Skip to content
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.

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

DB: Add product class count query #648

Merged
merged 2 commits into from
Feb 28, 2025

Conversation

ksharma-xyz
Copy link
Owner

@ksharma-xyz ksharma-xyz commented Feb 28, 2025

TL;DR

Added a new method to count product classes in the NSW stops database.

What changed?

  • Added productClassCount() method to the Sandook interface
  • Implemented the method in RealSandook class
  • Created new SQL query selectStopProductClassCount to count entries in the NswStopProductClass table

How to test?

  1. Call productClassCount() on a Sandook instance
  2. Verify the returned count matches the number of entries in the NswStopProductClass table
  3. Add/remove product classes and confirm the count updates accordingly

Why make this change?

To provide a way to monitor and verify the number of product classes stored in the database, which is useful for data validation and debugging purposes.

@ksharma-xyz ksharma-xyz force-pushed the 02-24-prioritisestops_based_on_priority_order branch from 86bc7e1 to 51a1318 Compare February 28, 2025 06:52
@ksharma-xyz ksharma-xyz changed the base branch from 02-24-prioritisestops_based_on_priority_order to graphite-base/648 February 28, 2025 06:54
@ksharma-xyz ksharma-xyz force-pushed the 02-28-add_selectstopproductclasscount branch from 7bdb4fc to d6d0110 Compare February 28, 2025 06:54
@ksharma-xyz ksharma-xyz changed the base branch from graphite-base/648 to 02-25-bump_to_1.3_version_-_getting_ready_for_alpha_testing February 28, 2025 06:54
@ksharma-xyz ksharma-xyz marked this pull request as ready for review February 28, 2025 06:55
@ksharma-xyz ksharma-xyz changed the title Add selectStopProductClassCount DB: Add product class count query Feb 28, 2025
@ksharma-xyz ksharma-xyz force-pushed the 02-25-bump_to_1.3_version_-_getting_ready_for_alpha_testing branch from 9a458cc to 27abaa5 Compare February 28, 2025 07:04
@ksharma-xyz ksharma-xyz force-pushed the 02-28-add_selectstopproductclasscount branch 2 times, most recently from 2a56b9f to c2d4750 Compare February 28, 2025 08:02
@ksharma-xyz ksharma-xyz force-pushed the 02-25-bump_to_1.3_version_-_getting_ready_for_alpha_testing branch from 27abaa5 to 07514f0 Compare February 28, 2025 08:02
@ksharma-xyz ksharma-xyz force-pushed the 02-25-bump_to_1.3_version_-_getting_ready_for_alpha_testing branch from 07514f0 to 3ea4217 Compare February 28, 2025 08:15
@ksharma-xyz ksharma-xyz force-pushed the 02-28-add_selectstopproductclasscount branch from c2d4750 to aeb5c9f Compare February 28, 2025 08:16
@ksharma-xyz ksharma-xyz force-pushed the 02-25-bump_to_1.3_version_-_getting_ready_for_alpha_testing branch 2 times, most recently from 89c1cc9 to dcfb505 Compare February 28, 2025 08:33
@ksharma-xyz ksharma-xyz force-pushed the 02-28-add_selectstopproductclasscount branch from aeb5c9f to 3d31b23 Compare February 28, 2025 08:33
@ksharma-xyz ksharma-xyz changed the base branch from 02-25-bump_to_1.3_version_-_getting_ready_for_alpha_testing to graphite-base/648 February 28, 2025 08:45
@ksharma-xyz ksharma-xyz force-pushed the 02-28-add_selectstopproductclasscount branch from 3d31b23 to 19172fe Compare February 28, 2025 08:45
@ksharma-xyz ksharma-xyz changed the base branch from graphite-base/648 to main February 28, 2025 08:45
@ksharma-xyz ksharma-xyz force-pushed the 02-28-add_selectstopproductclasscount branch from 19172fe to 4af4496 Compare February 28, 2025 08:45
@ksharma-xyz ksharma-xyz force-pushed the 02-28-add_selectstopproductclasscount branch from 4af4496 to 676b2ca Compare February 28, 2025 09:52
@ksharma-xyz ksharma-xyz merged commit a24765d into main Feb 28, 2025
2 checks passed
Copy link
Owner Author

Merge activity

  • Feb 28, 5:05 AM EST: A user merged this pull request with Graphite.

@ksharma-xyz ksharma-xyz deleted the 02-28-add_selectstopproductclasscount branch February 28, 2025 10:05
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