Skip to content

OpenConceptLab/ocl_online#517 | treat pro as community - #945

Closed
snyaggarwal wants to merge 1 commit into
masterfrom
ocl_online#517
Closed

snyaggarwal wants to merge 1 commit into
masterfrom
ocl_online#517

Conversation

@snyaggarwal

@snyaggarwal snyaggarwal commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Linked Issue

Ref OpenConceptLab/ocl_online#517

siblings
OpenConceptLab/oclmap#105
OpenConceptLab/oclweb3#66

@snyaggarwal snyaggarwal left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Verdict: Approve

Issue complete for the oclapi2 scope: yes. pro now maps to the early-access capacity tier, existing staff/core precedence is preserved, and the focused tier test covers the new group.

Security blockers: none.
Testing gap: the preferred API container is not running locally, so I relied on the focused test added by the PR and the PR checks.

GitHub does not allow authors to formally approve their own PR, so this verdict is posted as a review comment.

(snyaggarwal • Codex/GPT-5)

@paynejd paynejd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

recommend closing this and not merging

@snyaggarwal

Copy link
Copy Markdown
Contributor Author

Closing without merging. Plan names and the limits they grant stay out of open code, so this PR's approach is withdrawn. The reworked approach is tracked on the private ticket (ocl_online#517).

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.

2 participants