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

feat(relayer): HTTP api for exposing events table for bridge UI #271

Merged
merged 6 commits into from
Nov 23, 2022

Conversation

cyberhorsey
Copy link
Contributor

@cyberhorsey cyberhorsey commented Nov 14, 2022

Closes #245

WIP. It is probably better to expose a websockets endpoint rather than have the client poll. Maybe HTTP for the Events themselves and WSS for status updates.

@vercel
Copy link

vercel bot commented Nov 14, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated
website ⬜️ Ignored (Inspect) Nov 14, 2022 at 8:12PM (UTC)

@cyberhorsey cyberhorsey self-assigned this Nov 14, 2022
@cyberhorsey cyberhorsey changed the title fear(relayer): WIP: on HTTP api for exposing events table for bridge UI feat(relayer): DRAFT: on HTTP api for exposing events table for bridge UI Nov 14, 2022
@codecov
Copy link

codecov bot commented Nov 23, 2022

Codecov Report

Merging #271 (3098248) into main (5fdfdfa) will increase coverage by 0.18%.
The diff coverage is 74.41%.

@@            Coverage Diff             @@
##             main     #271      +/-   ##
==========================================
+ Coverage   62.91%   63.10%   +0.18%     
==========================================
  Files          82       83       +1     
  Lines        2416     2450      +34     
  Branches      281      281              
==========================================
+ Hits         1520     1546      +26     
- Misses        828      832       +4     
- Partials       68       72       +4     
Flag Coverage Δ *Carryforward flag
protocol 58.57% <ø> (ø) Carriedforward from 18966fc
relayer 69.02% <74.41%> (+0.24%) ⬆️

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files Coverage Δ
packages/relayer/message/process_message.go 67.54% <0.00%> (ø)
packages/relayer/http/get_events_by_address.go 50.00% <50.00%> (ø)
packages/relayer/cli/cli.go 61.27% <73.33%> (+1.63%) ⬆️
packages/relayer/http/routes.go 100.00% <100.00%> (ø)
packages/relayer/http/server.go 96.00% <100.00%> (-1.02%) ⬇️
packages/relayer/indexer/handle_event.go 50.00% <100.00%> (ø)
packages/relayer/repo/event.go 75.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@cyberhorsey cyberhorsey changed the title feat(relayer): DRAFT: on HTTP api for exposing events table for bridge UI feat(relayer): HTTP api for exposing events table for bridge UI Nov 23, 2022
@cyberhorsey cyberhorsey merged commit 7b5e6b8 into main Nov 23, 2022
@cyberhorsey cyberhorsey deleted the relayer_api branch November 23, 2022 15:52
@github-actions github-actions bot mentioned this pull request Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[relayer] expose API for events table for UI
3 participants