Skip to content

feat: added create more functionality#2214

Merged
ItzNotABug merged 9 commits into
spreadsheetfrom
feat-SER-259-adding-create-rows-functionality
Aug 22, 2025
Merged

feat: added create more functionality#2214
ItzNotABug merged 9 commits into
spreadsheetfrom
feat-SER-259-adding-create-rows-functionality

Conversation

@HarshMN2345
Copy link
Copy Markdown
Member

@HarshMN2345 HarshMN2345 commented Aug 12, 2025

What does this PR do?

This PR adds a "Create more" toggle feature to the table row creation sidebar

Test Plan

image

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

yes

@appwrite
Copy link
Copy Markdown

appwrite Bot commented Aug 12, 2025

Console

Project ID: 688b7bf400350cbd60e9

Sites (2)
Site Status Logs Preview QR
 console-qa
688b7cf6003b1842c9dc
Queued Queued View Logs Preview URL QR Code
 console-cloud
688b7c18002b9b871a8f
Queued Queued View Logs Preview URL QR Code

Note

Appwrite has a Discord community with over 16 000 members.

@HarshMN2345 HarshMN2345 marked this pull request as draft August 12, 2025 13:36
@HarshMN2345 HarshMN2345 marked this pull request as ready for review August 14, 2025 06:01
@HarshMN2345 HarshMN2345 requested a review from ItzNotABug August 14, 2025 06:01
@HarshMN2345 HarshMN2345 self-assigned this Aug 14, 2025
}
}
}}
{footer}>
Copy link
Copy Markdown
Contributor

@ItzNotABug ItzNotABug Aug 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we already add a snippet footer above, right?
afaik, we should directly be able to use the footer snippet inside the SideSheet, same with children.

@HarshMN2345 HarshMN2345 requested a review from ItzNotABug August 17, 2025 15:18
await invalidate(Dependencies.ROW);

if (createMore) {
createRow.set(computeInitialCreateRow());
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets make a separate function resetCreateRow or invalidateCreateRow, etc.

@HarshMN2345 HarshMN2345 requested a review from ItzNotABug August 18, 2025 06:40
text: string;
disabled?: boolean;
onClick?: () => void | Promise<void>;
onClick?: () => boolean | void | Promise<boolean | void>;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think with the addition of logic to always return true or false for keeping the sidesheet open, is there a usecase where we return void?

@HarshMN2345 HarshMN2345 requested a review from ItzNotABug August 22, 2025 12:00
@HarshMN2345 HarshMN2345 requested a review from ItzNotABug August 22, 2025 12:37
@ItzNotABug ItzNotABug merged commit 068af61 into spreadsheet Aug 22, 2025
2 of 5 checks passed
@ItzNotABug ItzNotABug deleted the feat-SER-259-adding-create-rows-functionality branch August 22, 2025 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants