Skip to content

Conversation

the1Riddle
Copy link
Collaborator

A readme and script on how to connect your HTML forms with google Spreadsheet for data transmission and more.

@the1Riddle the1Riddle added documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers labels Apr 13, 2025
@the1Riddle the1Riddle requested a review from Copilot April 13, 2025 20:21
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

Send-Html_Form-to-Spreadsheet/README.md:80

  • [nitpick] There is an inconsistency in the sheet name used in this code sample ('Sheet1') versus the one in the main script ('Ivi'). Consider aligning the sheet names or clarifying this difference in the documentation to avoid confusion.
        const sheet = SpreadsheetApp.openByUrl("https://docs.google.com/spreadsheets/d/.../edit?usp=sharing").getSheetByName("Sheet1");

@the1Riddle the1Riddle requested a review from Copilot April 13, 2025 20:28
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (1)

Send-Html_Form-to-Spreadsheet/code.gs:11

  • [nitpick] Clarify the comment by replacing "int's" with "integers" to improve readability.
            // here is where you will add your rows: avoid strings for int's

@the1Riddle the1Riddle requested a review from Copilot April 13, 2025 20:35
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (2)

Send-Html_Form-to-Spreadsheet/code.gs:11

  • [nitpick] Consider revising the comment to 'avoid strings for ints' to improve clarity.
 // here is where you will add your rows: avoid strings for int's

Send-Html_Form-to-Spreadsheet/README.md:80

  • The README references a sheet named 'Sheet1' while the Apps Script in code.gs uses 'Ivi'. Consider aligning the sheet names to avoid confusion and potential runtime errors.
const sheet = SpreadsheetApp.openByUrl("https://docs.google.com/spreadsheets/d/.../edit?usp=sharing").getSheetByName("Sheet1");

Comment on lines +13 to +14
data["example-name"], // example Name
data["example-phone"], // example Phone
Copy link
Preview

Copilot AI Apr 13, 2025

Choose a reason for hiding this comment

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

The parameter key 'example-name' does not match the expected 'name' as shown in the README code sample. This mismatch may lead to unexpected behavior when processing form submissions.

Suggested change
data["example-name"], // example Name
data["example-phone"], // example Phone
data["name"], // example Name

Copilot uses AI. Check for mistakes.

Comment on lines +13 to +14
data["example-name"], // example Name
data["example-phone"], // example Phone
Copy link
Preview

Copilot AI Apr 13, 2025

Choose a reason for hiding this comment

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

The parameter key 'example-phone' is inconsistent with the expected fields in the README (which uses 'email' and 'message'). Update the parameter keys to align with the form data expected per the documentation.

Suggested change
data["example-name"], // example Name
data["example-phone"], // example Phone
data["email"], // Email
data["message"], // Message

Copilot uses AI. Check for mistakes.

@the1Riddle the1Riddle merged commit 479a7a4 into main Apr 13, 2025
1 check passed
@the1Riddle the1Riddle deleted the gsheet branch April 13, 2025 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant