composing email with attachments fails on Android (fix provided)
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Accessibilité débutants
- 50/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- android, javascript
- Domaine
- mobile
Piste de recherche
Ouvrez index.android.js et comparez la gestion des pièces jointes avec la boucle for-of dans index.ios.js. Vérifiez la modification d’itération demandée et testez la composition d’un e-mail avec un tableau de pièces jointes sur Android ; le travail est terminé lorsque les pièces jointes ne déclenchent plus l’erreur indexOf signalée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
@nativescript/Email v2.0.5
Error
Calling email.compose with array 'attachments' fails with this error:
JS: 'Error in email.compose: TypeError: Cannot read property 'indexOf' of undefined'
Problem
The problem is that 'compose' in "index.android.js" parses attachments using a "for-in" loop. My array created via brackets in JS has a 'move' method by default, and so this for-in loop parses 'attachments[move]' which fails.
Fix
The fix is simply to change it to a "for-of" loop, as is the case in "index.ios.js".
Replace this...
for (const a in arg.attachments) {
const attachment = arg.attachments[a];
with this...
for (const attachment of arg.attachments) {
- Langage dominant
- TypeScript
- Étoiles
- 206
- Forks
- 123
- Merge moyen
- 2 j 1 h
- PR mergées (30 j)
- 1
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de NativeScript/plugins
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 92/100
NativeScript/plugins#680 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
NativeScript/plugins#671 ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
NativeScript/plugins#640 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
NativeScript/plugins#668 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
NativeScript/plugins#665 · 1 commentaire ·
Toutes les issues de NativeScript/plugins
Issues similaires
-
comp/dashboard P3 type/bug
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
NousResearch/hermes-agent#117722 ·
-
clawsweeper:fix-shape-clear clawsweeper:queueable-fix clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster no-stale P3
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficulté 1/5 Moins d'une heure Accessibilité débutants 76/100
-
code-quality refactoring
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
github/gh-aw-firewall#8816 ·
-
integration:quickjs org:external priority:backlog topic:code-interpreter topic:middleware type:feature
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
langchain-ai/deepagents#6450 ·