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

Initial AP and Reuters poller infra #93

Merged
merged 4 commits into from
Jan 13, 2025
Merged

Conversation

bryophyta
Copy link
Contributor

@bryophyta bryophyta commented Dec 23, 2024

What does this change?

  • Add minimal definition of Reuters and AP poller lambdas, using the patterns added in poller-lambdas #83:
  • Fixes the poller lambdas index.ts export, so that the lambdas can find the right function to run: 548ac91

Getting this merged into main will make the branch deployable to CODE again, and the actual business logic for the two pollers can then be added at a later point.

How to test

  • Successful deployment to CODE.

How can we measure success?

Have we considered potential risks?

Images

Accessibility

@bryophyta bryophyta force-pushed the pf/default-handler-export branch from 4c36f25 to 548ac91 Compare December 23, 2024 16:14
@bryophyta bryophyta changed the title Named export for poller lambda handlers Initial AP and Reuters poller infra Jan 13, 2025
@bryophyta bryophyta force-pushed the pf/default-handler-export branch from c7ee601 to 37d35e4 Compare January 13, 2025 12:01
@bryophyta bryophyta force-pushed the pf/default-handler-export branch from 37d35e4 to 9c2816c Compare January 13, 2025 12:03
@bryophyta bryophyta marked this pull request as ready for review January 13, 2025 12:23
@bryophyta bryophyta requested a review from a team as a code owner January 13, 2025 12:23
@bryophyta bryophyta mentioned this pull request Jan 13, 2025
7 tasks
@bryophyta bryophyta merged commit 93e01d9 into main Jan 13, 2025
3 checks passed
@bryophyta bryophyta deleted the pf/default-handler-export branch January 13, 2025 15:27
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