Conversation
components added
| const { id } = defineProps({ | ||
| id: { type: String, required: true }, | ||
| }) | ||
|
|
There was a problem hiding this comment.
Simplification des expressions booléennes, ternaires non utiles
app/components/Viewer/TreeObject.vue
Outdated
|
|
||
| await Promise.all( | ||
| removed.map((item) => dataStyleStore.setVisibility(item.id, false)), | ||
| ) |
There was a problem hiding this comment.
Promise.all pour s'assurer que toutesles mises à jour de visibilité sont terminées avant de faire le rendu
| const internal_files = ref(props.files) | ||
| const auto_upload = ref(props.auto_upload) | ||
| const internal_files = ref(files) | ||
| const internal_auto_upload = ref(auto_upload) |
There was a problem hiding this comment.
renommage de la ref interne pour éviter la confusion avec la prop
| } | ||
| const promise_array = internal_files.value.map((file) => | ||
| upload_file({ route: schema.$id, file }), | ||
| ) |
There was a problem hiding this comment.
Supprime l'enrobagee des promesses redondantes puisque upload_file en retourne déjà une.
.map() et Promise.all rend le traitement des fichiers asynchrone et élimine la gestion manuelle des réussites/échecs dans une boucle
| await Promise.all( | ||
| stores.map(async (store) => { | ||
| if (!store.exportStores) return | ||
| const storeId = store.$id | ||
| try { | ||
| snapshot[storeId] = await store.exportStores(params) | ||
| exportCount += 1 | ||
| } catch (error) { | ||
| console.error(`[AppStore] Error exporting store "${storeId}":`, error) | ||
| } | ||
| }), | ||
| ) |
There was a problem hiding this comment.
Combinaison de Promise.all et map() pour se passer de la boucle
|
🎉 This PR is included in version 10.3.1-rc.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
3 participants
No description provided.