Google Sheets #1156
Merged
Google Sheets #1156
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Tagging OptionsShould a new tag be published when this PR is merged?
|
…s type from unknown[][] to any[][] per Google API spec - Fix range pattern to accept single cells (A1) and ranges (A1:B10) - Remove unsupported @examples and @Schema annotations - Simplify value validation to match Google API requirements - Clean up documentation and remove redundant comments - Improve type safety with deno-lint-ignore for any types
… Create @types.ts with user-friendly simplified types - Add mappers.ts with bidirectional conversion utilities - Update updateValues and batchUpdateValues to use simplified interface - Add comprehensive validation and error handling - Improve documentation with Portuguese descriptions and examples - Types are now reusable across all Google Sheets actions
- Remove incorrect @types.ts file placement - Migrate all type definitions to utils/types.ts - Create utils/rangeUtils.ts for reusable range manipulation functions - Create utils/mappers.ts for API mapping utilities - Translate all comments and JSDoc from Portuguese to English - Clean up code by removing unnecessary comment separators - Improve code organization with centralized utilities
- Simplify batchUpdateValues and updateValues props structure - Replace complex range parameters with simple sheet_name + values - Add automatic range calculation from data dimensions - Implement first_cell_location parameter (defaults to A1) - Add comprehensive validation with clear error messages - Maintain backward compatibility while improving usability - Focus on essential parameters for easier LLM integration
- Add CREATE_SPREADSHEET annotation to createSpreadsheet.ts - Add OAUTH_CALLBACK annotation to oauth/callback.ts - Improve action discoverability and organization - Standardize semantic naming convention across actions
- Add GET_SPREADSHEET_VALUES annotation to getValues.ts - Add GET_BATCH_SPREADSHEET_VALUES annotation to getBatchValues.ts - Add GET_SPREADSHEET_METADATA annotation to getSpreadsheet.ts - Add OAUTH_START annotation to oauth/start.ts - Translate remaining Portuguese comments to English - Improve loader discoverability and semantic organization
- Fix import order according to Deno formatting rules - Apply consistent line length and spacing - Improve code formatting for better readability - Ensure all files follow Deno style conventions
- Make clientId and clientSecret non-optional in OAuthProvider interface - Add fallback empty strings for clientId and clientSecret in mod.ts - Ensure better type safety for OAuth configuration - Fix potential runtime issues with undefined OAuth credentials
- Change clientId and clientSecret from optional to required fields - Improve type safety for OAuth provider configuration - Prevent runtime errors with undefined OAuth credentials
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What is this Contribution About?
Please provide a brief description of the changes or enhancements you are proposing in this pull request.
Issue Link
Please link to the relevant issue that this pull request addresses:
Loom Video
Demonstration Link