v0.5.35: helm updates, copilot improvements, 404 for docs, salesforce fixes, subflow resize clamping#2455
v0.5.35: helm updates, copilot improvements, 404 for docs, salesforce fixes, subflow resize clamping#2455icecrasher321 merged 21 commits intomainfrom
Conversation
…state, fix order of operations for copilot edit workflow, add sleep tool (#2440) * Fix copilot ooo * Add copilot sleep tool * Fix lint
* fix(condition): async execution isolated vm error * fix tests
…ed confirmation for disable auth FF (#2447) * fix(auth): added same-origin validation to forget password route, added confirmation for disable auth FF * ack PR comments
…enerateurl function when the useWebhookurl flag is true
…n, developer, and custom domain salesforce orgs (#2441) (#2444) * fix(oauth): updated oauth providers that had unstable reference IDs leading to duplicate oauth records (#2441) * fix(oauth): updated oauth providers that had unstable reference IDs leading to duplicate oauth records * ack PR comments * ack PR comments * cleanup salesforce refresh logic * ack more PR comments
* feat(docs): added 404 page for the docs * added metadata
…ookUrl-conditioning improvement(useWebhookUrl): GET api/webhook is called when useWebhookUrl:true
…inor consistency issue with filter parsing (#2452) * fix(notifs-slac): display name for account * fix inactivity polling check * consolidate trigger types * remove redundant defaults * fix
…ts (#2454) * improvement(helm): added SSO and cloud storage variables to helm charts * consolidated sf types
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Greptile Summary
Important Files Changed
Confidence score: 4/5
|
There was a problem hiding this comment.
Additional Comments (2)
-
apps/sim/tools/salesforce/get_accounts.ts, line 68-94 (link)style: complex JWT decoding logic is duplicated - consider using the imported
getInstanceUrlutilityNote: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
-
apps/sim/tools/salesforce/create_account.ts, line 125-157 (link)style: Complex inline instance URL extraction duplicates the imported
getInstanceUrlutility function. Should this logic use the imported getInstanceUrl utility instead?Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
66 files reviewed, 3 comments
* fix(docs): clarify working directory for drizzle migration (#2375) * fix(landing): prevent url encoding for spaces for footer links (#2376) * fix: handle empty body.value in Teams webhook notification parser (#2425) * Update directory path for migration command --------- Co-authored-by: Vikhyath Mondreti <vikhyathvikku@gmail.com> Co-authored-by: Waleed <walif6@gmail.com> Co-authored-by: waleedlatif1 <waleedlatif1@users.noreply.github.com> Co-authored-by: icecrasher321 <icecrasher321@users.noreply.github.com> Co-authored-by: Siddharth Ganesan <33737564+Sg312@users.noreply.github.com> Co-authored-by: mosa <mosaxiv@gmail.com> Co-authored-by: Emir Karabeg <78010029+emir-karabeg@users.noreply.github.com> Co-authored-by: Adam Gough <77861281+aadamgough@users.noreply.github.com> Co-authored-by: Shivam <shivamprajapati035@gmail.com> Co-authored-by: Gaurav Chadha <65453826+Chadha93@users.noreply.github.com> Co-authored-by: root <root@Delta.localdomain>
…flow (#2448) * Add edge handle validation * Clean * Fix lint * Fix empty target handle
Co-authored-by: priyanshu.solanki <priyanshu.solanki@saviynt.com>
Uh oh!
There was an error while loading. Please reload this page.