Skip to content

Commit

Permalink
Merge branch 'trunk' of github.com:woocommerce/woocommerce-android in…
Browse files Browse the repository at this point in the history
…to ci/use-custom-init-gradle-task-to-download-dependencies
  • Loading branch information
ParaskP7 committed Jan 29, 2025
2 parents d950365 + ca38ed6 commit 20c13f9
Show file tree
Hide file tree
Showing 27 changed files with 1,329 additions and 311 deletions.
1 change: 1 addition & 0 deletions RELEASE-NOTES.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
*** For entries which are touching the Android Wear app's, start entry with `[WEAR]` too.
21.6
-----
- [**] The UK customers have access to the POS now [https://github.com/woocommerce/woocommerce-android/pull/13386]
- [Internal] [*] Fixed a bug that could lead to a crash when formatting dates and times [https://github.com/woocommerce/woocommerce-android/pull/13393]

21.5
Expand Down
1 change: 0 additions & 1 deletion WooCommerce-Wear/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,6 @@ dependencies {
// Testing
testImplementation(libs.junit)
testImplementation(libs.mockito.kotlin)
testImplementation(libs.mockito.inline)
testImplementation(libs.kotlin.test.junit)
testImplementation(libs.assertj.core)
testImplementation(libs.androidx.arch.core.testing) {
Expand Down
1 change: 0 additions & 1 deletion WooCommerce/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -331,7 +331,6 @@ dependencies {
// Dependencies for local unit tests
testImplementation(libs.junit)
testImplementation(libs.mockito.kotlin)
testImplementation(libs.mockito.inline)
testImplementation(libs.kotlin.test.junit)
testImplementation(libs.assertj.core)
testImplementation(libs.androidx.arch.core.testing) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ class ProductImagesNotificationHandler @Inject constructor(
.setDestination(R.id.mediaUploadErrorsFragment)
.setArguments(
MediaUploadErrorListFragmentArgs(
remoteId = productId,
remoteProductId = productId,
errorList = errors.toTypedArray()
).toBundle()
)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -346,7 +346,7 @@ class MainActivity :

if (savedInstanceState == null) {
viewModel.handleIncomingAppLink(intent?.data)
viewModel.handleShortcutAction(intent?.action?.toLowerCase(Locale.ROOT))
viewModel.handleShortcutAction(intent?.action?.lowercase(Locale.ROOT))
handleIncomingImages()
}
}
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,59 +1,37 @@
package com.woocommerce.android.ui.media

import android.os.Bundle
import android.view.LayoutInflater
import android.view.View
import android.view.ViewGroup
import androidx.fragment.app.viewModels
import androidx.navigation.fragment.findNavController
import androidx.recyclerview.widget.LinearLayoutManager
import com.woocommerce.android.R
import com.woocommerce.android.analytics.AnalyticsTracker
import com.woocommerce.android.databinding.FragmentMediaUploadErrorListBinding
import com.woocommerce.android.extensions.takeIfNotEqualTo
import com.woocommerce.android.ui.base.BaseFragment
import com.woocommerce.android.ui.compose.composeView
import com.woocommerce.android.ui.main.AppBarStatus
import com.woocommerce.android.util.setupTabletSecondPaneToolbar
import com.woocommerce.android.viewmodel.MultiLiveEvent
import dagger.hilt.android.AndroidEntryPoint

@AndroidEntryPoint
class MediaUploadErrorListFragment : BaseFragment(R.layout.fragment_media_upload_error_list) {
class MediaUploadErrorListFragment : BaseFragment() {
override val activityAppBarStatus: AppBarStatus
get() = AppBarStatus.Hidden

private val mediaUploadErrorListAdapter: MediaUploadErrorListAdapter by lazy {
MediaUploadErrorListAdapter()
}

private val viewModel: MediaUploadErrorListViewModel by viewModels()

override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
super.onViewCreated(view, savedInstanceState)
val binding = FragmentMediaUploadErrorListBinding.bind(view)

with(binding.mediaUploadErrorList) {
adapter = mediaUploadErrorListAdapter
layoutManager = LinearLayoutManager(requireContext(), LinearLayoutManager.VERTICAL, false)
override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View {
return composeView {
MediaUploadErrorListScreen(viewModel)
}

setupObservers(viewModel)

setupTabletSecondPaneToolbar(
title = getString(R.string.product_downloadable_files_upload_failed),
onMenuItemSelected = { _ -> false },
onCreateMenu = { toolbar ->
toolbar.setNavigationOnClickListener {
findNavController().navigateUp()
}
}
)
}

private fun setupObservers(viewModel: MediaUploadErrorListViewModel) {
viewModel.viewStateData.observe(viewLifecycleOwner) { old, new ->
new.uploadErrorList.takeIfNotEqualTo(old?.uploadErrorList) { errors ->
mediaUploadErrorListAdapter.mediaErrorList = errors
}
new.toolBarTitle.takeIfNotEqualTo(old?.toolBarTitle) {
activity?.title = it
override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
super.onViewCreated(view, savedInstanceState)

viewModel.event.observe(viewLifecycleOwner) { event ->
when (event) {
is MultiLiveEvent.Event.Exit -> findNavController().navigateUp()
}
}
}
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,169 @@
package com.woocommerce.android.ui.media

import androidx.compose.foundation.background
import androidx.compose.foundation.layout.Arrangement
import androidx.compose.foundation.layout.Column
import androidx.compose.foundation.layout.Row
import androidx.compose.foundation.layout.fillMaxWidth
import androidx.compose.foundation.layout.height
import androidx.compose.foundation.layout.padding
import androidx.compose.foundation.layout.width
import androidx.compose.foundation.lazy.LazyColumn
import androidx.compose.foundation.lazy.items
import androidx.compose.foundation.shape.RoundedCornerShape
import androidx.compose.material.Icon
import androidx.compose.material.IconButton
import androidx.compose.material.MaterialTheme
import androidx.compose.material.Scaffold
import androidx.compose.material.Text
import androidx.compose.material.icons.Icons
import androidx.compose.material.icons.automirrored.filled.ArrowBack
import androidx.compose.runtime.Composable
import androidx.compose.runtime.livedata.observeAsState
import androidx.compose.ui.Alignment
import androidx.compose.ui.Modifier
import androidx.compose.ui.draw.clip
import androidx.compose.ui.layout.ContentScale
import androidx.compose.ui.platform.LocalContext
import androidx.compose.ui.res.painterResource
import androidx.compose.ui.res.stringResource
import androidx.compose.ui.tooling.preview.Preview
import androidx.compose.ui.unit.dp
import coil.compose.AsyncImage
import coil.request.ImageRequest
import com.woocommerce.android.R
import com.woocommerce.android.ui.compose.component.Toolbar
import com.woocommerce.android.ui.compose.theme.WooTypography
import com.woocommerce.android.ui.media.MediaUploadErrorListViewModel.ErrorUiModel

@Composable
fun MediaUploadErrorListScreen(viewModel: MediaUploadErrorListViewModel) {
viewModel.viewState.observeAsState().value?.let { state ->
MediaUploadErrorListScreen(
state = state,
viewModel::onBackPressed,
viewModel::onRetryUploadClicked
)
}
}

@Composable
private fun MediaUploadErrorListScreen(
state: MediaUploadErrorListViewModel.ViewState,
onBackPressed: () -> Unit,
onRetryClicked: (ErrorUiModel) -> Unit
) {
Scaffold(
topBar = {
Toolbar(
title = stringResource(id = R.string.product_upload_error_title),
onNavigationButtonClick = onBackPressed,
navigationIcon = Icons.AutoMirrored.Filled.ArrowBack,
)
},
content = { padding ->
MediaUploadErrorList(
errors = state.uploadErrorList,
onRetryClicked = onRetryClicked,
modifier = Modifier
.padding(padding)
.background(MaterialTheme.colors.surface)
.padding(horizontal = 16.dp)
)
}
)
}

@Composable
private fun MediaUploadErrorList(
errors: List<ErrorUiModel>,
onRetryClicked: (ErrorUiModel) -> Unit,
modifier: Modifier = Modifier
) {
LazyColumn(modifier = modifier.fillMaxWidth()) {
items(errors) { error ->
MediaUploadErrorListItem(
error = error,
onRetryClicked = onRetryClicked,
modifier = Modifier.animateItem()
)
}
}
}

@Composable
fun MediaUploadErrorListItem(
error: ErrorUiModel,
onRetryClicked: (ErrorUiModel) -> Unit,
modifier: Modifier = Modifier
) {
Row(
modifier = modifier
.fillMaxWidth()
.padding(vertical = 6.dp),
verticalAlignment = Alignment.CenterVertically
) {
AsyncImage(
model = ImageRequest.Builder(LocalContext.current)
.data(error.localUri)
.crossfade(true)
.build(),
fallback = painterResource(R.drawable.ic_product),
placeholder = painterResource(R.drawable.ic_product),
error = painterResource(R.drawable.ic_product),
contentDescription = stringResource(id = R.string.product_image_content_description),
contentScale = ContentScale.Crop,
modifier = Modifier
.width(48.dp)
.height(48.dp)
.clip(shape = RoundedCornerShape(size = 6.dp))
)
Column(
modifier = Modifier
.weight(1f)
.padding(8.dp),
verticalArrangement = Arrangement.spacedBy(6.dp)
) {
Text(
style = WooTypography.subtitle2,
text = error.fileName
)
Text(
style = WooTypography.body2,
text = error.errorMessage
)
}
IconButton(
onClick = { onRetryClicked(error) },
) {
Icon(
painter = painterResource(id = R.drawable.ic_gridicons_refresh),
contentDescription = stringResource(id = R.string.retry)
)
}
}
}

@Preview
@Composable
fun MediaUploadErrorListItemPreview() {
MediaUploadErrorListScreen(
state = MediaUploadErrorListViewModel.ViewState(
uploadErrorList = listOf(
ErrorUiModel(
fileName = "image1345211331.jpg",
errorMessage = "Upload timeout",
localUri = ""
),
ErrorUiModel(
fileName = "image987654.jpg",
errorMessage = "Upload failed, very long error message with multiple lines to check how" +
" the row behaves",
localUri = ""
)
)
),
onBackPressed = {},
onRetryClicked = {}
)
}
Loading

0 comments on commit 20c13f9

Please sign in to comment.