-
Notifications
You must be signed in to change notification settings - Fork 3.1k
v0.5.35: helm updates, copilot improvements, 404 for docs, salesforce fixes, subflow resize clamping #2455
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.