Skip to content

feat: Android SDK update for version 14.1.0#118

Merged
ChiragAgg5k merged 3 commits intomainfrom
dev
Mar 30, 2026
Merged

feat: Android SDK update for version 14.1.0#118
ChiragAgg5k merged 3 commits intomainfrom
dev

Conversation

@ChiragAgg5k
Copy link
Copy Markdown
Member

@ChiragAgg5k ChiragAgg5k commented Mar 30, 2026

This PR contains updates to the Android SDK for version 14.1.0.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 30, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: adbd9470-6585-4fba-afe3-489b03f6cb82

📥 Commits

Reviewing files that changed from the base of the PR and between f992599 and 5e9efcb.

📒 Files selected for processing (3)
  • CHANGELOG.md
  • README.md
  • library/src/main/java/io/appwrite/Client.kt
✅ Files skipped from review due to trivial changes (2)
  • CHANGELOG.md
  • README.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • library/src/main/java/io/appwrite/Client.kt

Walkthrough

Added a new 14.1.0 changelog entry and bumped published SDK coordinates in README to 14.1.0. Updated library POM developer email in build.gradle to team@localhost.test. In Client.kt the default x-sdk-version header was updated to 14.1.0 and three public accessors were added: getHeaders(): Map<String, String>, getCookies(url: String): List<Cookie>, and getHttpClient(): OkHttpClient.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: Android SDK update for version 14.1.0' accurately summarizes the main change: a version bump from 14.0.0 to 14.1.0 with three new public Client methods added to the Android SDK.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@greptile-apps
Copy link
Copy Markdown

greptile-apps bot commented Mar 30, 2026

Greptile Summary

This PR bumps the Android SDK to version 14.0.1, updating the version string in Client.kt, both dependency snippets in README.md, and adding a CHANGELOG.md entry. All of those changes are correct and consistent. However, library/build.gradle contains a problematic change: POM_DEVELOPER_EMAIL was switched from the legitimate team@appwrite.io to team@localhost.test, a non-routable placeholder address that will be embedded in the published Maven POM artifact.

  • library/src/main/java/io/appwrite/Client.ktx-sdk-version header correctly updated to 14.0.1
  • README.md — Gradle and Maven dependency versions correctly updated to 14.0.1
  • CHANGELOG.md — New 14.0.1 section added with appropriate entry
  • library/build.gradleP1: POM_DEVELOPER_EMAIL set to team@localhost.test, a test/placeholder address that should be reverted to team@appwrite.io before publishing

Confidence Score: 4/5

Not safe to merge as-is — the placeholder email will be published as permanent POM metadata to Maven repositories.

All version-bump changes are correct, but the POM_DEVELOPER_EMAIL value of 'team@localhost.test' is a non-functional placeholder that would be baked into the published Maven artifact's POM file. This is a P1 defect that should be fixed (reverting to team@appwrite.io) before the release is cut.

library/build.gradle — POM_DEVELOPER_EMAIL must be corrected before publishing

Important Files Changed

Filename Overview
library/build.gradle POM_DEVELOPER_EMAIL changed from 'team@appwrite.io' to 'team@localhost.test' — a non-functional placeholder that will be published in Maven POM metadata
library/src/main/java/io/appwrite/Client.kt SDK version header bumped from 14.0.0 to 14.0.1 — straightforward and correct
README.md Version references updated from 14.0.0 to 14.0.1 in both Gradle and Maven dependency snippets — correct
CHANGELOG.md New 14.0.1 changelog entry added — correct structure and placement

Reviews (1): Last reviewed commit: "chore: update Android SDK to 14.0.1" | Re-trigger Greptile

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@library/build.gradle`:
- Line 18: The POM_DEVELOPER_EMAIL variable is set to a non-routable test
address (POM_DEVELOPER_EMAIL = 'team@localhost.test') which will be written into
published POM metadata; update POM_DEVELOPER_EMAIL to a real maintainer or team
contact email (or remove/omit it for release) so the artifact metadata contains
a valid contact before publishing.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e6bc5d32-4349-4a5a-ae5f-445aa956ccbe

📥 Commits

Reviewing files that changed from the base of the PR and between c72d0d8 and f992599.

📒 Files selected for processing (4)
  • CHANGELOG.md
  • README.md
  • library/build.gradle
  • library/src/main/java/io/appwrite/Client.kt

@ChiragAgg5k ChiragAgg5k changed the title feat: Android SDK update for version 14.0.1 feat: Android SDK update for version 14.1.0 Mar 30, 2026
@ChiragAgg5k ChiragAgg5k merged commit 6e551b2 into main Mar 30, 2026
1 check passed
@ChiragAgg5k ChiragAgg5k deleted the dev branch March 30, 2026 05:25
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