Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enable back at contributors #1086

Merged

Conversation

furusin
Copy link
Contributor

@furusin furusin commented Sep 4, 2023

Issue

  • there is no Issue about this PR. I just found the difference.

Overview (Required)

Links

Screenshot (Optional if screenshot test is present or unrelated to UI)

Before After

Movie (Optional)

Before After
Screen_recording_20230904_232601.webm
Screen_recording_20230904_232419.webm

took the same solution with StaffScreen
@furusin furusin requested a review from a team as a code owner September 4, 2023 14:29
@furusin furusin requested a review from mhidaka September 4, 2023 14:29
@github-actions
Copy link

github-actions bot commented Sep 4, 2023

Test Results

215 tests   215 ✔️  7m 5s ⏱️
  11 suites      0 💤
  11 files        0

Results for commit 027d785.

♻️ This comment has been updated with latest results.

@github-actions github-actions bot temporarily deployed to deploygate-distribution September 4, 2023 14:54 Inactive
@@ -78,6 +79,7 @@ private fun ContributorsScreen(
null
}
val localLayoutDirection = LocalLayoutDirection.current
val lifecycleOwner = LocalLifecycleOwner.current
Copy link
Member

Choose a reason for hiding this comment

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

Good catch! Thanks for finding this issue!
But we can't use this function in commonMain because this function is used by iOS 🙏

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, My! Ok, I'll fix it

@github-actions github-actions bot temporarily deployed to deploygate-distribution September 8, 2023 15:23 Inactive
@takahirom
Copy link
Member

Looks great! Thanks

@furusin
Copy link
Contributor Author

furusin commented Sep 9, 2023

@takahirom
Thank you for your check! Please merge whenever you can.

@takahirom takahirom merged commit 22562a1 into DroidKaigi:main Sep 10, 2023
8 checks passed
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.

None yet

2 participants