Skip to content

AJO Bigtext Template Support#86

Open
ritusingh-29 wants to merge 12 commits into
adobe:mainfrom
ritusingh-29:feature/ajo-bigtext-product-templates
Open

AJO Bigtext Template Support#86
ritusingh-29 wants to merge 12 commits into
adobe:mainfrom
ritusingh-29:feature/ajo-bigtext-product-templates

Conversation

@ritusingh-29

@ritusingh-29 ritusingh-29 commented Jun 10, 2026

Copy link
Copy Markdown

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

ritusingh-29 added 10 commits June 4, 2026 23:52
@ritusingh-29
Add AJO Basic push notification template support with scale type
1502827
@ritusingh-29
Apply Spotless formatting
358c4ab
@ritusingh-29
Add unit tests for AJO basic push template and notification builder
fd8b43e
@ritusingh-29
refactor(ajo_basic): merge expanded layouts and honor large icon scal…
48e16d6 
…e type

- Collapse two scale-type expanded XMLs into ajo_basic_push_template_expanded.xml
- Use visibility toggling for centerCrop/fitCenter on both expanded image and large icon
- Update collapsed layout to use large_icon_container FrameLayout pattern
- Delete ajo_push_template_expanded_center_crop.xml and ajo_push_template_expanded_fit_center.xml
@ritusingh-29
add missing tests
edc764b
@ritusingh-29
fix(tests): add missing import and AJO_BIG_TEXT dispatch test for CI …
b45858b 
…coverage
@ritusingh-29
fix(tests): remove bigtext refs and fix missing import in AJOBasicPus…
1ec0ba6 
…hTemplateTest
@ritusingh-29
test: cover unknown carousel type else branch in NotificationBuilder
b167a9f
@ritusingh-29
fix: sort CarouselPushTemplate import to satisfy Spotless
780a626
@ritusingh-29
fix: remove untestable carousel else branch test
9ec1c8e
@codecov

codecov Bot commented Jun 10, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.86777% with 10 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ternal/ajo/builders/AJOBasicNotificationBuilder.kt 96.15% 0 Missing and 3 partials ⚠️
...rnal/ajo/builders/AJOBigTextNotificationBuilder.kt 96.00% 0 Missing and 3 partials ⚠️
...builder/internal/templates/AJOBasicPushTemplate.kt 95.00% 0 Missing and 2 partials ⚠️
...ilder/internal/templates/AJOBigTextPushTemplate.kt 94.28% 0 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

@ritusingh-29 ritusingh-29 force-pushed the feature/ajo-bigtext-product-templates branch from 15cf294 to e865d86 Compare June 22, 2026 20:20
@ritusingh-29 ritusingh-29 force-pushed the feature/ajo-bigtext-product-templates branch from e865d86 to 51b75da Compare June 22, 2026 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ritusingh-29