Skip to content

feat(discovery-service): add fallback for chain state, e2e test in the starknet integration environment - #466

Merged
m-kus merged 1 commit into
mainfrom
m-kus/no-websocket-fallback
Feb 23, 2026
Merged

m-kus merged 1 commit into
mainfrom
m-kus/no-websocket-fallback

Conversation

@m-kus

@m-kus m-kus commented Feb 10, 2026 •

Copy link
Copy Markdown

This change is Reviewable

@m-kus
m-kus force-pushed the m-kus/no-websocket-fallback branch from 0084758 to 0daa517 Compare February 13, 2026 20:28
@m-kus
m-kus changed the base branch from 02-07-feat_indexer_e2e_discovery_flow to pr-split/17-e2e-tests February 13, 2026 20:28
@m-kus
m-kus force-pushed the m-kus/no-websocket-fallback branch 2 times, most recently from bfe90c8 to 69c34af Compare February 13, 2026 20:52
@m-kus
m-kus marked this pull request as ready for review February 13, 2026 20:52
@m-kus
m-kus force-pushed the m-kus/no-websocket-fallback branch from 69c34af to 36d81ee Compare February 13, 2026 21:10
@m-kus
m-kus force-pushed the m-kus/no-websocket-fallback branch from 36d81ee to 9ae8e0c Compare February 15, 2026 14:45
@m-kus
m-kus force-pushed the pr-split/17-e2e-tests branch from d466433 to 840db55 Compare February 16, 2026 15:15
@m-kus
m-kus force-pushed the m-kus/no-websocket-fallback branch from 9ae8e0c to 60ae788 Compare February 16, 2026 15:15
Comment thread e2e/package-lock.json
@m-kus
m-kus force-pushed the pr-split/17-e2e-tests branch from 840db55 to e8c4a80 Compare February 16, 2026 16:21

@Yoni-Starkware Yoni-Starkware left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Yoni-Starkware reviewed 13 files and all commit messages.
Reviewable status: 13 of 14 files reviewed, 1 unresolved discussion (waiting on m-kus).

@m-kus
m-kus force-pushed the pr-split/17-e2e-tests branch from 6b809f4 to 6f97132 Compare February 22, 2026 17:26
@m-kus
m-kus force-pushed the m-kus/no-websocket-fallback branch from 29fd0cb to 7014b5f Compare February 22, 2026 17:26
@m-kus
m-kus force-pushed the pr-split/17-e2e-tests branch from 6f97132 to 9be52eb Compare February 22, 2026 18:19
@m-kus
m-kus force-pushed the m-kus/no-websocket-fallback branch from 7014b5f to bf4f735 Compare February 22, 2026 18:19

@Yoni-Starkware Yoni-Starkware left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@Yoni-Starkware reviewed 5 files and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on m-kus).

@m-kus
m-kus force-pushed the m-kus/no-websocket-fallback branch from bf4f735 to 831c0d7 Compare February 23, 2026 12:05
@m-kus
m-kus force-pushed the pr-split/17-e2e-tests branch from 9be52eb to 4cb1d28 Compare February 23, 2026 12:05

@Yoni-Starkware Yoni-Starkware left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Yoni-Starkware reviewed 1 file and all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on m-kus).

@m-kus
m-kus changed the base branch from pr-split/17-e2e-tests to graphite-base/466 February 23, 2026 14:37
@m-kus
m-kus force-pushed the graphite-base/466 branch from 4cb1d28 to 956dda4 Compare February 23, 2026 15:17
@m-kus
m-kus force-pushed the m-kus/no-websocket-fallback branch from 831c0d7 to 02defd1 Compare February 23, 2026 15:17
@m-kus
m-kus changed the base branch from graphite-base/466 to main February 23, 2026 15:17
@m-kus
m-kus force-pushed the m-kus/no-websocket-fallback branch from 02defd1 to 5aa2ef8 Compare February 23, 2026 15:19
Comment thread e2e/package-lock.json
@m-kus
m-kus merged commit b32b02d into main Feb 23, 2026
7 of 8 checks passed
@m-kus
m-kus deleted the m-kus/no-websocket-fallback branch February 23, 2026 15:37
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