Skip to content

Chore/sick abroad deflect GEN-5133#2914

Merged
panasetskaya merged 8 commits intodevelopfrom
chore/sickAbroadDeflect
Apr 8, 2026
Merged

Chore/sick abroad deflect GEN-5133#2914
panasetskaya merged 8 commits intodevelopfrom
chore/sickAbroadDeflect

Conversation

@panasetskaya
Copy link
Copy Markdown
Contributor

@panasetskaya panasetskaya commented Apr 7, 2026

  • replace deprecated sickAbroadAction with sickAbroadDeflect, setting preferredMinHeight to null for now
  • todo: move ExtendedPartnersScreen and SimplePartnersScreen from claim chat feature to shared and use it in help center and claim chat

@panasetskaya panasetskaya requested a review from a team as a code owner April 7, 2026 14:00
@panasetskaya panasetskaya marked this pull request as draft April 7, 2026 14:05
@panasetskaya panasetskaya changed the title Chore/sick abroad deflect Chore/sick abroad deflect GEN-5133 Apr 8, 2026
@notion-workspace
Copy link
Copy Markdown

@panasetskaya panasetskaya marked this pull request as ready for review April 8, 2026 11:06
val content: InfoBlock,
val faq: List<InfoBlock>,
val buttonText: String,
@Contextual
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't actually know how this works, is it still fine with the navigation library and all?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

seems to work! IDE asked for it

implementation(projects.composeUi)
}
androidMain.dependencies {
implementation(libs.androidx.compose.foundation)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Without testing it, I am assuming that this should still work even without this androidMain block at all if we bring in the foundation dependency in commonMain. Did you encounter issues without it or is this just a leftover from a migration?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I had problems with my preview and added android to plugin and this too, and it got better:) will remove dep part

@panasetskaya panasetskaya merged commit e855b58 into develop Apr 8, 2026
4 checks passed
@panasetskaya panasetskaya deleted the chore/sickAbroadDeflect branch April 8, 2026 13:11
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.

3 participants