Principiante aquí...
Así que quería hacer lo que pensé que era un cambio simple, pero resultó ser más difícil. Tengo un script .js que se ejecuta en un temporizador como función. La función consulta datos y trae una lista de objetos que se convierten en una matriz de ID de registros en el primer bucle.
En su formato original -
// get pledge objects const pledges = await getOppPledges() //Check if there are records if (pledges && pledges.length > 0) { //get array of pledge opportunity ids for (let pledge of pledges) { let oppIds = pledges.map(p => p.Opportunity__c) let sfOpps = await getOpp(oppIds) //run loop to create oppty doc for await (let opp of sfOpps) {... } ...}Mi problema es que el ciclo doble "for" genera documentos dobles cuando hay más de un compromiso. Cuando intenté cerrar el primer ciclo justo antes del segundo ciclo, "sfOpps" no estaba disponible para el segundo ciclo y el código se rompió.
Había reorganizado el código a algo como,
if (pledges && pledges.length > 0) { for (let pledge of pledges) { let oppIds = [pledge.Opportunity__c] let opp = await getOpp(oppIds)Pero parece que devuelve "opp" como una matriz en lugar de un objeto. Esto tampoco funciona porque vuelve como indefinido cuando solicito un valor de "opp" como "opp.OpportunityContactRoles.totalSize".
¡Cualquier entrada es apreciada!
Si entendí tu problema correctamente, no deberías necesitar el primer ciclo ya que mapeas tus identificaciones en todas tus promesas dentro del ciclo. Es la razón por la que haces las operaciones n veces. (siendo n la longitud de la matriz "promesas")
let oppIds = pledges.map(p => p.Opportunity__c) // You're already mapping all the ids at onceLa forma correcta de hacerlo sería
// get pledge objects const pledges = await getOppPledges() //Check if there are records if (pledges && pledges.length > 0) { //get array of pledge opportunity ids let oppIds = pledges.map(p => p.Opportunity__c) let sfOpps = await getOpp(oppIds) //run loop to create oppty doc for await (let opp of sfOpps) {... }