Finding which tier an organization holds on a ladder is a loop written five times #188
Closed
opened 2026-10-06 21:25:46 +00:00 by cgalo5758
·
1 comment
Labels
Clear labels
accessibility
area/billing
area/catalog
area/discourse
area/domains
area/entitlements
area/fedwiki
area/identity
area/integrations
area/licensing
area/member-ui
area/meta
area/operator-ui
area/ops
area/testing
duplicate
good-first-issue
invalid
privacy
security
upstream
wontfix
A barrier for people using assistive technology or a keyboard alone.
The Stripe mirror, checkout, subscriptions, invoices, fulfillment.
Products, prices, plan ladders, purchasability.
The Discourse integration.
The domains registry, claims, placements, the certificate ask.
Entitlement sets, rules, grants, pools, provisioning.
The Federated Wiki integration and farm sync.
Sign-in, sessions, persons, organizations, workspaces, roles.
The provider registry, outbox and webhooks in general.
Licenses, the contributor agreement, SPDX headers.
Member pages.
The repository itself, its contributing guide, CI, the tracker and the workflow.
Operator pages, forms, lists, the design system.
Deployment, configuration, migrations, workflows, instance settings.
The test stack, screens, lint, walkthroughs.
Closed because another issue already covers it.
Small, self-contained, and explained enough to be a first contribution.
Closed because it is not a ticket for this repository.
Touches what a person's data reveals.
Touches authentication, authorization, secrets or data exposure.
Waits on another repository or project before it can move.
Closed because it will not be done, with the reason in the last comment.
kind
bug
The software does something other than what it promises; closed when it again does what it promises.
kind
debt
Code, tests or tooling to clean up with nothing visible changing; closed when they are cleaner.
kind
design
A question to settle before work can be defined; closed when the decision is written down.
kind
docs
Documentation that is wrong or missing; closed when it says the right thing.
kind
enhancement
Something the software does not do yet; closed when it does.
priority
critical
Blocks the active milestone or harms members now.
priority
high
Next in line inside the active milestone.
priority
low
Inside the active milestone, when nothing else is left.
priority
medium
Inside the active milestone, after the high ones.
Milestone
No items
No Milestone
Public launch
Projects
Clear projects
No projects
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: wiki-cafe/member-console#188
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
The loop over an organization's pools and their active attachments is written four times in
internal/server/member_products.goand once ininternal/fulfillment/plan_change.go. Two copies keep the first attachment and two the last, and all walk every pool though an organization has one. The Products page reads the same provision up to five times and the same subscription three times per ladder. #148 has to change every copy.Done when: one query on the default pool returns, per ladder, the tier, its provision and source, and a subscription's period and scheduled change; nothing else loops over pools; and the page reads each row once.
The Products page, Checkout, the plan switch and its preview, the entitlements panel, the dashboard's entitlement check and fulfillment's plan changes now read an organization's position on each ladder from one query,
ListLadderPositionsByPool(internal/entitlements/queries/pool_provision_ladders.sql), on its default pool. Each provision and subscription is read once per request. The reads that still repeat are a product's prices, mapping and rules, once per ladder the product sits on.The operator side still loops over an organization's pools in
internal/server/operator_enrollment.go: the Pools panel inloadOrgEnrollmentData,IssueGrant, andsubscriptionHeldLaddersbehind the Issue grant form's warning. They move in the operator reads step of #166.