pipedream
pipedream copied to clipboard
Oanda - fix weekday name
The upstream API expects it to be spelled correctly
Summary by CodeRabbit
-
Bug Fixes
- Corrected a spelling error in weekly alignment configuration ("Thursday").
-
Chores
- Bumped package release to 0.7.1.
- Updated version metadata for several OANDA components and actions.
✏️ Tip: You can customize this high-level summary in your review settings.
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
| Project | Deployment | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| pipedream-docs-redirect-do-not-edit | Dec 4, 2025 0:11am |
Thank you so much for submitting this! We've added it to our backlog to review, and our team has been notified.
Thanks for submitting this PR! When we review PRs, we follow the Pipedream component guidelines. If you're not familiar, here's a quick checklist:
- [ ] Create components to address specific use cases whenever possible
- [ ] Component
keys should follow the formatapp_name_slug-slugified-component-name - [ ] Components should follow the standard directory structure
- [ ] Prefer Node.js client libraries to REST APIs
- [ ] When making API requests, handle pagination to ensure all data / events are processed
- [ ] Use
secretprops to capture sensitive data - [ ] Props and methods should be defined in app files whenever possible
- [ ] Document methods with JS Docs
- [ ] Use
optionalprops whenever possible, and set adefaultvalue where you can - [ ] Use async options to accept user input wherever possible
Walkthrough
Package and component metadata versions were incremented across several OANDA action/source modules and the package, and a typo in WEEKLY_ALIGNMENT_DAYS was corrected.
Changes
| Cohort / File(s) | Summary |
|---|---|
Package version components/oanda/package.json |
Bumped package version from 0.7.0 to 0.7.1. |
Action/source metadata version bumps components/oanda/actions/get-historical-prices/get-historical-prices.mjs, components/oanda/actions/create-order/create-order.mjs, components/oanda/actions/list-trades/list-trades.mjs, components/oanda/sources/new-transaction/new-transaction.mjs |
Updated exported version fields: actions/source metadata bumped from 0.0.1/0.0.2 to 0.0.2/0.0.3 as applicable (no logic changes). |
Constants fix components/oanda/common/constants.mjs |
Fixed typo in WEEKLY_ALIGNMENT_DAYS: "Thrusday" → "Thursday" (no other logic changes). |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
- Pay attention to
components/oanda/common/constants.mjsto confirm no other typos or related data issues. - Quick sanity-check of the version-only edits in action/source files to ensure no accidental code changes were included.
Pre-merge checks and finishing touches
✅ Passed checks (3 passed)
| Check name | Status | Explanation |
|---|---|---|
| Title check | ✅ Passed | The title 'Oanda - fix weekday name' clearly and specifically describes the main change: fixing a misspelled weekday name in the OANDA component. |
| Description check | ✅ Passed | The description explains the reason for the fix (upstream API expects correct spelling) and references the OANDA API documentation, though it does not follow the repository's template structure. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
✨ Finishing touches
- [ ] 📝 Generate docstrings
🧪 Generate unit tests (beta)
- [ ] Create PR with unit tests
- [ ] Post copyable unit tests in a comment
📜 Recent review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro
📥 Commits
Reviewing files that changed from the base of the PR and between 5773462b323211effa8c1c51fb81ad5220f834dd and 5d0aef0746752e742ea0c10023fa52be2ec1cef6.
📒 Files selected for processing (3)
components/oanda/actions/create-order/create-order.mjs(1 hunks)components/oanda/actions/list-trades/list-trades.mjs(1 hunks)components/oanda/sources/new-transaction/new-transaction.mjs(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
- GitHub Check: pnpm publish
- GitHub Check: Publish TypeScript components
- GitHub Check: Verify TypeScript components
- GitHub Check: Lint Code Base
🔇 Additional comments (3)
components/oanda/actions/list-trades/list-trades.mjs (1)
8-8: LGTM!Version bump appropriately aligns this action with the constants spelling fix.
components/oanda/actions/create-order/create-order.mjs (1)
8-8: LGTM!Version bump appropriately aligns this action with the constants spelling fix.
components/oanda/sources/new-transaction/new-transaction.mjs (1)
10-10: LGTM!Version bump appropriately aligns this source with the constants spelling fix.
Comment @coderabbitai help to get the list of available commands and usage tips.
Hi, this is just a typo fix so I'll move this to ready for release