Skip to content

chore: fix README links, add LICENSE - #87

Open
Vlada-Q wants to merge 2 commits into
qonversion:mainfrom
Vlada-Q:marketing/hygiene-2026-09
Open

Vlada-Q wants to merge 2 commits into
qonversion:mainfrom
Vlada-Q:marketing/hygiene-2026-09

Conversation

@Vlada-Q

@Vlada-Q Vlada-Q commented Sep 22, 2026 •

Copy link
Copy Markdown

Two broken links, both from the shared README template.

  • [All other integrations here](qonversion.io/integrations) has no scheme. GitHub renders it as a repository-relative path, so on the rendered README it resolves to github.com/qonversion/capacitor-plugin/blob/main/qonversion.io/integrations and 404s. Now https://qonversion.io/integrations (200).
  • https://documentation.qonversion.io/discuss-new in Have a question? is left over from readme.io and returns 404 — it is not among the redirects the docs site keeps. That is the link a reader follows when something did not work. It now points at https://qonversion.io/contact-us (200), next to the existing GitHub issues link. The same target is used in the other SDK READMEs so the support channel is one and the same everywhere.

The GitHub issues link in the same block still used the repository's former name (qonversion-capacitor) and only resolved through GitHub's rename redirect — updated to the current name.

No licence text in the repository. The README ends with «Qonversion SDK is available under the MIT license» and carries an MIT badge, but there is no LICENSE file and the GitHub API reports license: null for this repository — so formally the licence exists only as a claim. For a team running a legal review before adopting an SDK, that is a stopper.

Added LICENSE with the MIT text copied from qonversion-ios-sdk, copyright holder unchanged (Qonversion team <hi@qonversion.io>). The badge now links to that file instead of the #license anchor inside the README.


Scope is deliberately narrow: links, licence, install, typo. Positioning, the opening paragraph, the bullets, the banner images and the version badge are untouched — those belong to the README template being rewritten separately.

Opened from a fork, so the review request could not be set from here — @SpertsyanKM, this one is for you.

🤖 Generated with Claude Code

Vlada-Q and others added 2 commits September 22, 2026 14:56
`[All other integrations here](qonversion.io/integrations)` has no scheme,
so GitHub renders it as a repository-relative path and it 404s where the
README is actually read. `https://documentation.qonversion.io/discuss-new`
is left over from readme.io and 404s too — it sits in "Have a question?",
which is exactly where a reader goes when something did not work. It now
points at the contact form on the site, next to the existing issues link.

The issues link also still used the repository's former name and only
resolved through GitHub's rename redirect.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The README states the SDK is available under the MIT license and carries
an MIT badge, but the repository has no licence text in any form and the
GitHub API reports `license: null`. The text is the one used in
qonversion-ios-sdk, unchanged. The badge now links to the file instead of
an anchor inside the README.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: qonversion/coderabbit/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: bee060c1-b697-4108-803c-b5258bbbcf1f

📥 Commits

Reviewing files that changed from the base of the PR and between 38a5654 and 8775d74.

📒 Files selected for processing (2)
  • LICENSE
  • README.md
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • qonversion/go-micro (manual)

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The pull request adds an MIT LICENSE file and updates three README links for the license, integrations, and contact options.

Changes

License and Documentation

Layer / File(s) Summary
License and README link updates
LICENSE, README.md
The repository adds the MIT license text. The README license badge, integrations link, issue link, and contact link now use the specified destinations.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 8775d

The change adds the MIT license and corrects README destinations without affecting runtime behavior, so it is mergeable.

🚥 Pre-merge checks | ✅ 8
✅ Passed checks (8 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Behavior And Regression Safety ✅ Passed The PR changes only README links and adds the MIT LICENSE file. The new local badge target exists, and the changed integrations, contact, and GitHub issues URLs returned HTTP 200. The diff has no whit…
Security And Tenant Isolation ✅ Passed PASS. The pull request changes only README.md links and adds a static MIT LICENSE file. It does not change authentication, authorization, tenant scope, secrets, personal data, command or SQL construct…
Contract And Data Compatibility ✅ Passed The review-scoped diff changes only README.md and adds LICENSE. It changes documentation links and adds MIT license text. It does not change an API, event, database, configuration, queue, or serialize…
Rollout And Recovery Evidence ✅ Passed The pull request changes only README links and adds a LICENSE file. It does not change production behavior, configuration, infrastructure, or migrations. The rollout and recovery evidence requirement …
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title identifies the affected documentation and licensing areas and states the outcomes: fixing README links and adding the LICENSE file. It matches the pull request changes.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

Comment @coderabbitai help to get the list of available commands.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant