Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/log text messages #2

Merged
merged 3 commits into from
Jul 4, 2023
Merged

Feature/log text messages #2

merged 3 commits into from
Jul 4, 2023

Conversation

Plong-Wasin
Copy link
Owner

@Plong-Wasin Plong-Wasin commented Jul 4, 2023

Add log text message

…to improve readability and adhere to naming conventions

✨ feat(Code.ts): add interfaces for JsonRequest, ChatEvent, DeliveryContext, Message, ContentProvider, and Source to improve type safety and code organization
🐛 fix(Code.ts): change variable name from event to jsonRequest in run() function to match parameter name
🐛 fix(Code.ts): change variable name from event to jsonRequest in writeMessageLog() function to match parameter name
🐛 fix(Code.ts): change variable name from event to jsonRequest in getEvent() function to match parameter name
🐛 fix(Code.ts): change variable name from event to jsonRequest in saveFile() function to match parameter name
🐛 fix(Code.ts): change variable name from event to jsonRequest in getLink() function to match parameter name
✨ feat(Code.ts): add function getSelectedId() to improve code readability and reduce duplication
✨ feat(Code.ts): add function getSelectedFolder() to improve code readability and reduce duplication
✨ feat(Code.ts): add function createSpreadsheetFromFolder() to create a new spreadsheet in a specified folder or retrieve an existing one
✨ feat(Code.ts): add function createSheetIfNotExistsFromSpreadsheetId() to create a new sheet in a spreadsheet if it doesn't already exist
🐛 fix(Code.ts): change variable name from event to jsonRequest in sendMsg() function to match parameter name
🐛 fix(Code.ts): change variable name from event to jsonRequest in downloadFileFromURL() function to match parameter name
✨ feat(Code.ts): add type annotation for selectedId parameter in getConfigValue() function to improve type safety
…mprove semantics

✨ feat(Code.ts): add interface 'LineUser' to define the structure of a Line user
🐛 fix(Code.ts): change function name 'writeMessageLog' to 'writeMessageLog' to improve readability
✨ feat(Code.ts): add support for writing message logs to a spreadsheet
🐛 fix(Code.ts): change function name 'createSpreadsheetFromFolder' to 'createSpreadsheetFromFolderId' to improve semantics
✨ feat(Code.ts): add support for creating a spreadsheet from a folder ID
🐛 fix(Code.ts): change function name 'createSheetIfNotExistsFromSpreadsheet' to 'createSheetIfNotExistsFromSpreadsheetId' to improve semantics
✨ feat(Code.ts): add support for creating a sheet if it doesn't exist in a spreadsheet by ID
✨ feat(Code.ts): add support for getting the Line user profile information
✨ feat(Code.ts): add support for extracting links from a string
…from clasp commands

🐛 fix(Code.ts): move writeMessageLog() call inside the condition to check if SAVE_MESSAGE is true to prevent unnecessary calls

🔀 merge(Code.ts): add SAVE_MESSAGE configuration option to defaultConfig() function to enable/disable saving messages
@Plong-Wasin Plong-Wasin merged commit d34f193 into main Jul 4, 2023
@Plong-Wasin Plong-Wasin mentioned this pull request Jul 4, 2023
@Plong-Wasin Plong-Wasin deleted the feature/log-text-messages branch July 4, 2023 18:21
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.

None yet

1 participant