Skip to content

fix: State loss in search screen during backstack restoration#85

Merged
komodgn merged 1 commit intodevelopfrom
fix/backstack
Jan 17, 2026
Merged

fix: State loss in search screen during backstack restoration#85
komodgn merged 1 commit intodevelopfrom
fix/backstack

Conversation

@komodgn
Copy link
Owner

@komodgn komodgn commented Jan 17, 2026

Summary by CodeRabbit

  • Chores
    • Updated Circuit dependency to the latest version.

✏️ Tip: You can customize this high-level summary in your review settings.

Description

This PR addresses a state loss bug occurring during back navigation from the PhotoDetail screen.

Technical Details

• Upgraded Circuit to v0.32.0.
• This update includes a fix for the rememberRetained state resetting issue previously reported in the library.

Reference

slackhq/circuit#2477

Check

Search results preserved

@coderabbitai
Copy link

coderabbitai bot commented Jan 17, 2026

📝 Walkthrough

Walkthrough

The Circuit dependency version has been updated from 0.31.0 to 0.32.0 in the Gradle version catalog configuration file. This is a single dependency version bump with no other modifications to the build configuration.

Changes

Cohort / File(s) Summary
Dependency version update
gradle/libs.versions.toml
Circuit version bumped from 0.31.0 to 0.32.0 in [versions] section

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A version so fresh, from point-three-one to two,
Circuit flows smoother, with upgrades so new,
One line, one change, dependency true,
Building with vigor—hopping right through! 🔌✨

🚥 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 PR title in Korean describes fixing search data loss when returning from detail pages, but the only code change is a Circuit library version bump from 0.31.0 to 0.32.0, which is indirectly related but not the primary action taken.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


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.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🤖 Android CI Summary

Step Results:

  • Unit Test: ✅ Success (3m 19s)
  • Debug Build: ✅ Success (1m 9s)
  • Code Style Check: ✅ Success (3m 43s)

Total Time: 8m 11s

🎉 All steps completed successfully!

@komodgn komodgn merged commit 271a975 into develop Jan 17, 2026
2 checks passed
@komodgn komodgn deleted the fix/backstack branch January 17, 2026 07:45
@komodgn komodgn changed the title 상세 페이지에서 복귀 시 검색 데이터가 유실되는 오류를 수정한다 fix: State loss in search screen during backstack restoration Jan 18, 2026
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.

1 participant