Skip to content

chore(inbox): migrate messaginginbox to shared publish plugin#770

Merged
mahmoud-elmorabea merged 1 commit into
feat/overlay-inboxfrom
fix-messaginginbox-publish-plugin
Jul 6, 2026
Merged

chore(inbox): migrate messaginginbox to shared publish plugin#770
mahmoud-elmorabea merged 1 commit into
feat/overlay-inboxfrom
fix-messaginginbox-publish-plugin

Conversation

@mahmoud-elmorabea

@mahmoud-elmorabea mahmoud-elmorabea commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Fixes the Android sample-app build failure on feat/overlay-inbox after it was rebased onto latest main.

Root cause: main #738 adopted the shared io.customer.android.publish plugin and deleted scripts/publish-module.gradle. messaginginbox was created before #738 and still used apply from: "${rootDir}/scripts/publish-module.gradle", which no longer exists — aborting project evaluation before android-config.gradle sets compileSdk, so AGP failed with "project ':messaginginbox' does not specify compileSdk" and both sample apps failed to build.

Fix: migrate messaginginbox/build.gradle to apply plugin: 'io.customer.android.publish-module' + customerIoPublish { artifactId = ... }, matching every sibling module.

Draft, targeting feat/overlay-inbox. Merging this makes #769's sample-app build green.

🤖 Generated with Claude Code


Note

Low Risk
Build/publishing Gradle wiring only; no runtime or API changes in the inbox module.

Overview
Fixes sample-app evaluation for :messaginginbox after scripts/publish-module.gradle was removed on main (#738).

messaginginbox/build.gradle now uses apply plugin: 'io.customer.android.publish-module' and customerIoPublish { artifactId = "messaging-inbox" } instead of the deleted publish-module.gradle script and local PUBLISH_* ext properties. Maven coordinates stay the same; only the publishing setup is aligned with sibling modules.

Reviewed by Cursor Bugbot for commit 4ce8f1e. Bugbot is set up for automated code reviews on this repo. Configure here.

Rebasing feat/overlay-inbox onto main brought in #738 (adopt shared
io.customer.android.publish plugin), which deleted scripts/publish-module.gradle
and migrated every module to `apply plugin: 'io.customer.android.publish-module'`
+ `customerIoPublish { artifactId = ... }`. The messaginginbox module was created
on the feature branch before #738, so the rebase left it on the removed script,
aborting project evaluation before android-config.gradle set compileSdk and
failing the sample-app build. Migrate it to match its siblings (e.g. messaginginapp).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

Sample app builds 📱

Below you will find the list of the latest versions of the sample apps. It's recommended to always download the latest builds of the sample apps to accurately test the pull request.


@codecov

codecov Bot commented Jul 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (feat/overlay-inbox@ed05fb5). Learn more about missing BASE report.

Additional details and impacted files
@@                  Coverage Diff                  @@
##             feat/overlay-inbox     #770   +/-   ##
=====================================================
  Coverage                      ?   65.45%           
  Complexity                    ?     1018           
=====================================================
  Files                         ?      173           
  Lines                         ?     5848           
  Branches                      ?      874           
=====================================================
  Hits                          ?     3828           
  Misses                        ?     1719           
  Partials                      ?      301           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
  • kotlin_compose: fix-messaginginbox-publish-plugin (1783092071)

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
  • java_layout: fix-messaginginbox-publish-plugin (1783092074)

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

Build available to test
Version: fix-messaginginbox-publish-plugin-SNAPSHOT
Repository: https://central.sonatype.com/repository/maven-snapshots/

@mahmoud-elmorabea mahmoud-elmorabea marked this pull request as ready for review July 6, 2026 12:34
@mahmoud-elmorabea mahmoud-elmorabea requested a review from a team as a code owner July 6, 2026 12:34
@mahmoud-elmorabea mahmoud-elmorabea merged commit e65c84c into feat/overlay-inbox Jul 6, 2026
46 of 48 checks passed
@mahmoud-elmorabea mahmoud-elmorabea deleted the fix-messaginginbox-publish-plugin branch July 6, 2026 12:45
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