Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Task]: Remove audit log API call for JS action execution #33150

Closed
ankitakinger opened this issue May 3, 2024 · 0 comments · Fixed by #33151
Closed

[Task]: Remove audit log API call for JS action execution #33150

ankitakinger opened this issue May 3, 2024 · 0 comments · Fixed by #33151
Assignees
@ankitakinger
Labels
Audit Logs Audit trails to ensure data security Task A simple Todo Team Managers Pod Issues that team managers care about for the security and efficiency of their teams

Comments

@ankitakinger
Copy link
Contributor

SubTasks

Remove audit log API call for JS action execution
@ankitakinger ankitakinger added Task A simple Todo Audit Logs Audit trails to ensure data security labels May 3, 2024
@ankitakinger ankitakinger self-assigned this May 3, 2024
@github-actions github-actions bot added the Team Managers Pod Issues that team managers care about for the security and efficiency of their teams label May 3, 2024
@ankitakinger ankitakinger mentioned this issue May 3, 2024
Merged
2 tasks
ankitakinger added a commit that referenced this issue May 3, 2024
@ankitakinger
chore: Removing the audit log API call for JS action execution (#33151)
086582b 
## Description

Removing the audit log API call for JS action execution as discussed
here:
https://theappsmith.slack.com/archives/C02K0SZQ7V3/p1714112912735089

Fixes [#33150](#33150)

## Automation

/ok-to-test tags="@tag.All"

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!TIP]
> 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/8935334970>
> Commit: bb69ebf
> Cypress dashboard url: <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=8935334970&attempt=1"
target="_blank">Click here!</a>

<!-- end of auto-generated comment: Cypress test results  -->









## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [ ] No
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ankitakinger ankitakinger

Labels
Audit Logs Audit trails to ensure data security Task A simple Todo Team Managers Pod Issues that team managers care about for the security and efficiency of their teams
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: Removing the audit log API call for JS action execution appsmithorg/appsmith
1 participant
@ankitakinger