script to backup logs

[ayusham001]

/claim #367

[netlify]

✅ Deploy Preview for webcapgo ready!

Name	Link
🔨 Latest commit	c278ec5
🔍 Latest deploy log	https://app.netlify.com/sites/webcapgo/deploys/65150bf5e3ec2b000887fbde
😎 Deploy Preview	https://deploy-preview-369--webcapgo.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 78 (🔴 down 6 from production) Accessibility: 89 (no change from production) Best Practices: 92 (no change from production) SEO: 85 (no change from production) PWA: 100 (no change from production) View the detailed breakdown and full score reports

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[github-actions]

🤖 ChatGPT score: Based on the provided diff, I would score the pull request with a 90. The script adds functionality to backup and delete entries from specific tables in a Supabase database based on a cutoff date. The code appears well-structured and handles potential errors.

[rishi-raj-jain]

1. @ayusham001 Can you add a GitHub Action Workflow as well (as per Make a script to backup logs (stats table) and delete from db logs older than X days #367 (comment))?

2. Also, https://supabase.com/docs/reference/javascript/select#:~:text=%3D%20await%20query-,Fetch%20data,-Perform%20a%20SELECT supabase only returns top 1K rows, can you handle pagination as well?

[ayusham001]

Hi @rishi-raj-jain and @akhill10, I have made the necessary changes. Please take a look at it.

[rishi-raj-jain]

@ayusham001 can you resolve the comments from above you think are done? I still see some of them are not resolved after your latest commit.

[ayusham001]

@ayusham001 can you resolve the comments from above you think are done? I still see some of them are not resolved after your latest commit.

Which one are you talking about? I made some changes in the recent one

[rishi-raj-jain]

Here are the summary of changes that I think are still pending:

[akhill10]

@rishi-raj-jain It looks like you haven't submitted your review as it still shows pending in you're screenshots.

[ayusham001]

Here are the summary of changes that I think are still pending:

@rishi-raj-jain I guess you haven't submitted the review
because it's not showing here.

[rishi-raj-jain]

@ayusham001 I've submitted my review now. Thanks for informing!

[ayusham001]

@rishi-raj-jain I have made the changes, Please have a look at it.

[ayusham001]

Hi @riderx, I have made the changes that have been suggested
please have a look at it.

[rishi-raj-jain]

@ayusham001 taking a look right now

[riderx]

@ayusham001 did you evaluate to do it in SQL directly ? I'm afraid to bloat the db with the loop query that way

[rishi-raj-jain]

I've left a comment to be addressed ^^

Do confirm if you see it.

[ayusham001]

@rishi-raj-jain please have a look at it:)

[ayusham001]

@riderx I have used SQL query instead of Supabase SDK in my recent commit.
Please review it:)

[akhill10]

@ayusham001 did you evaluate to do it in SQL directly ? I'm afraid to bloat the db with the loop query that way

I think @riderx was pointing out that, instead of using supabase SDK. Create a SQL function in postgres db that executes this logic directly maybe with the help of storage extensions.

[rishi-raj-jain]

I think @riderx was pointing out that, instead of using supabase SDK. Create a SQL function in postgres db that executes this logic directly maybe with the help of storage extensions.

How is this different from using Supabase SQL rpc?

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

No Coverage information
0.0% Duplication

[rishi-raj-jain]

LGTM