AutoGPT icon indicating copy to clipboard operation
AutoGPT copied to clipboard

feat(market) Add on delete to market

Open aarushik93 opened this issue 1 year ago โ€ข 2 comments
trafficstars

Background

So we can easily delete agents in the marketplace in the db and have it cascade through to related tables

Changes ๐Ÿ—๏ธ

Added onDelete: Cascade to AnalyticsTracker and Agents

Testing ๐Ÿ”

[!NOTE] Only for the new autogpt platform, currently in autogpt_platform/

  • Create from scratch and execute an agent with at least 3 blocks
  • Import an agent from file upload, and confirm it executes correctly
  • Upload agent to marketplace
  • Import an agent from marketplace and confirm it executes correctly
  • Edit an agent from monitor, and confirm it executes correctly

aarushik93 avatar Oct 03 '24 12:10 aarushik93

PR Reviewer Guide ๐Ÿ”

Here are some key observations to aid the review process:

โฑ๏ธย Estimated effort to review: 2 ๐Ÿ”ต๐Ÿ”ตโšชโšชโšช
๐Ÿงชย No relevant tests
๐Ÿ”’ย No security concerns identified
โšกย Recommended focus areas for review

Potential Data Loss
The addition of 'onDelete: Cascade' to the AnalyticsTracker relation might lead to unintended data loss when deleting agents. Consider if this is the desired behavior.

Relationship Change
The relationship between Agents and AnalyticsTracker has been modified. Ensure this change is intentional and doesn't break existing queries or data integrity.

qodo-merge-pro[bot] avatar Oct 03 '24 12:10 qodo-merge-pro[bot]

Deploy Preview for auto-gpt-docs canceled.

Name Link
Latest commit 64532b8eb78774b964d76e5622e3e1e00dd89b77
Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/66ffb46faa29010008799e68

netlify[bot] avatar Oct 03 '24 12:10 netlify[bot]