feat: logic for archiving dataroom

[Khaan25]

This PR fixes: #785

I've included a boolean in the function to archive a dataroom.

Will discuss some key points with @mfts and finish this feature.

[vercel]

@Khaan25 is attempting to deploy a commit to the mftsio Team on Vercel.

A member of the Team first needs to authorize it.

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[mfts]

@Khaan25 looks good so far

• missing a field on Dataroom table for isArchived
• also missing the tab for the archived datarooms

[Khaan25]

Yes, I'm gonna work on it

[Khaan25]

Hey, In the Link table, we've isArchived field, is that for any other usecase?

[Khaan25]

Here's a video: https://streamable.com/0yl2ym

Could not upload more than 10 MBs of video on GitHub :)
FYI, Video link expires after 2 days :)

[Khaan25]

Just a quick question,

You can see the image above, if we're on trial and we create dataroom then the trial-banner day isn't showing because it's fetching from the first data-room tab in the image, how about we proceed?

My initial thoughts are, query for first tab, if null then query for second tab, that way we won't have "days left" but "X days left"

What do you say?

[mfts]

@Khaan25

My initial thoughts are, query for first tab, if null then query for second tab, that way we won't have "days left" but "X days left"

Don't worry about that trial banner. it's a minor thing

[Khaan25]

Alright. So we should mark this issue as complete?

[mfts]

@Khaan25 please sign the CLA as indicated above

[Khaan25]

/recheck

[Khaan25]

recheck

[Khaan25]

I have read the CLA Document and I hereby sign the CLA

[Khaan25]

@mfts I've signed it

[mfts]

Some minor changes, see inline comments.

Ideally we want to reduce code duplication

[mfts]

I think this is a redundant endpoint.

Better to add a "archived" query to GET /datarooms endpoint and then based on that return either all archived or non-archived documents.

Especially since this endpoint is not returning any more information than the index.ts

[mfts]

This needs to be replicated on /view/domains/[domain]/[slug]/index.ts

[Khaan25]

What's the URL or a button to view view/domains/[domain]/[slug] route?

[Khaan25]

Sure, I'll do it and let you know

[Khaan25]

@mfts I've pushed the changes, can you please review?

[mfts]

@Khaan25 looks good. I will make some small changes today and then push it

[Khaan25]

Cool

[Khaan25]

@mfts, Could you please review and give points? :)

[mfts]

/award 200

still requires changes from my end

[oss-gg]

Awarding Khaan25: 200 points 🕹️ Well done! Check out your new contribution on oss.gg/Khaan25