Add O'Saasy license#227
Open
zainfathoni wants to merge 6 commits intomainfrom
Open
Conversation
- Add Kamal config for kelas.pejuangkode.com - Add GitHub Actions workflow for pejuangkode deployment - Add branding config (SITE_NAME, SITE_DOMAIN, etc.) - Add course config for different content per site - Update components to use configurable branding - Update email service to use dynamic branding - Update login page with dynamic logo - Update .env.example with new branding variables Deployment: Kamal 2.0 to VPS 103.235.75.227 Domain: kelas.pejuangkode.com (DNS setup by Zain)
- Remove unnecessary quotes from YAML string values in config/deploy.pejuangkode.yml to satisfy the yml/plain-scalar ESLint rule (SITE_NAME, SITE_DOMAIN, SITE_URL, EMAIL_FROM) - Add explicit click() before fill() in phone number validation E2E test to ensure the focus event fires reliably in webkit, setting the 'interacted' React state needed for client-side validation error display Amp-Thread-ID: https://ampcode.com/threads/T-019c5ead-c672-72de-a79a-ed33d773553f Co-authored-by: Amp <amp@ampcode.com>
- Replace skipped unit test (it.skip) with proper test using createRemixStub with loader data for HomePage component - Enable allowConditional for playwright/no-skipped-test rule so conditional test.skip(isStagingEnv) is permitted - Merge main into feat/pejuangkode to resolve outdated branch Amp-Thread-ID: https://ampcode.com/threads/T-019c5eda-dc52-76d6-8892-c85919042771 Co-authored-by: Amp <amp@ampcode.com>
Protects the project from being cloned as competing SaaS while keeping code public. Ref: Card #201
![kodiakhq[bot]](https://avatars.githubusercontent.com/in/29196?s=60&v=4){kind=small}
zainfathoni
commented
Feb 19, 2026
Owner
Author
zainfathoni
left a comment
zainfathoni
left a comment
There was a problem hiding this comment.
O'Saasy License Review
The LICENSE file text correctly matches the canonical O'Saasy license from fizzy.do/license — nice choice for protecting against SaaS clones while keeping the code public.
However, there are two places that still reference MIT and need to be updated for consistency:
package.json—"license": "MIT"needs to be changed (e.g., to"SEE LICENSE IN LICENSE"or"LicenseRef-OSaasy"since O'Saasy isn't an SPDX identifier).README.md— The MIT license badge (license-MIT-blue) and[license]link still say MIT. The PR description mentions updating these but the diff doesn't include README changes.
Also a couple of observations:
- The LICENSE file doesn't name itself as "O'Saasy License" anywhere — adding that as a header would help discoverability.
- This PR bundles the entire pejuangkode multi-site feature (branding config, deployment workflow, hero refactoring, etc.) with the license change. Consider splitting into separate PRs for cleaner history, or at minimum updating the PR title/description to reflect the full scope.
Amp-Thread-ID: https://ampcode.com/threads/T-019c7a5d-a670-732a-a749-b64bac3aba45 Co-authored-by: Amp <amp@ampcode.com>
wheeljackz
reviewed
Feb 20, 2026
Collaborator
wheeljackz
left a comment
wheeljackz
left a comment
There was a problem hiding this comment.
Code Review: PR #227 — Add O'Saasy License
Summary
This PR changes the license from MIT to O'Saasy License (a source-available license with an anti-SaaS-compete clause) and updates README badges accordingly. It's stacked on top of PR #226 (env-based branding).
Analysis
License Change (LICENSE)
- ✅ The O'Saasy License text is clear: permissive like MIT but with an anti-compete clause for hosted/SaaS offerings
- ✅ Copyright year updated to 2026
⚠️ Consider: This is a significant licensing change. All existing contributors agreed to MIT. You may want to note this in the PR description or get acknowledgment from contributors (there are 6 listed in README)⚠️ O'Saasy is not OSI-approved — worth being explicit about this in README so users/contributors know it's source-available, not open-source
README Changes
- ✅ Badge correctly updated to show O'Saasy
Other Changes
- This PR carries all changes from #226 (branding). The license change itself is just the
LICENSEandREADME.mdfiles. Consider rebasing to keep the diff clean once #226 merges.
Verdict
The license change itself is clean. Main concern is the social/legal aspect of relicensing an existing MIT project with 6 contributors. Make sure contributors are aware.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds the O'Saasy license to protect the project from being cloned as a competing SaaS while keeping the code public.
Changes:
Context:
The O'Saasy license (https://www.fizzy.do/license) by 37signals is essentially MIT but prohibits using the software to directly compete by offering it as a hosted SaaS product.
Related: Fizzy Card #201