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.

Sign up for GitHub

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

Jump to bottom

feat: OPinit bot #10

Merged
merged 38 commits into from
Jan 23, 2024
Merged

feat: OPinit bot #10

merged 38 commits into from
Jan 23, 2024

Conversation

JSHan94
Copy link
Collaborator

@JSHan94 JSHan94 commented Nov 17, 2023

  • use base64 for all buffer (use initita.js > 0.1.15)
  • need cleanup for unused code

@JSHan94 JSHan94 requested a review from beer-1 November 17, 2023 08:11
beer-1
beer-1 requested changes Nov 20, 2023
View reviewed changes
bots/src/lib/query.ts Outdated Show resolved Hide resolved
bots/src/lib/lcd.ts Outdated Show resolved Hide resolved
bots/src/worker/batchSubmitter/batchSubmitter.ts Outdated Show resolved Hide resolved
bots/src/worker/bridgeExecutor/MonitorHelper.ts Show resolved Hide resolved
bots/src/worker/bridgeExecutor/MonitorHelper.ts Show resolved Hide resolved
bots/src/worker/challenger/challenger.ts Outdated Show resolved Hide resolved
bots/src/worker/challenger/challenger.ts Outdated Show resolved Hide resolved
bots/src/worker/challenger/challenger.ts Outdated Show resolved Hide resolved
bots/src/worker/challenger/challenger.ts Outdated Show resolved Hide resolved
bots/src/worker/challenger/challenger.ts Outdated Show resolved Hide resolved
beer-1
beer-1 requested changes Nov 20, 2023
View reviewed changes
Copy link
Contributor

@beer-1 beer-1 left a comment

Choose a reason for hiding this comment

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

Overall works are good, but need some changes in challenger

and new feature request for output submitter

beer-1

This comment was marked as duplicate.

Sign in to view

@beer-1 beer-1 force-pushed the main branch 2 times, most recently from a239c68 to 1656530 Compare November 22, 2023 08:43
@JSHan94 JSHan94 requested a review from beer-1 November 24, 2023 05:10
@JSHan94 JSHan94 force-pushed the feat/sdk branch 2 times, most recently from 11f1496 to e0d3e59 Compare December 13, 2023 11:03
@JSHan94
remove docker setup
7006080 
wip

tested version

use base64 for buffer and impelemnt challenger

fix challenger

handle pagination output proposal

clean up

update bots readme

remove unnecessary file

fix readme

fix base port in readme

fix ports in sample and config.ts

update initia.js

apply blockchainData

fix to handle block

fix sync error

activate L2 Monitor

fix handle event

support stone-12-1

update dockerfile

implement resurrector

remove mnemonic
beer-1
beer-1 reviewed Dec 15, 2023
View reviewed changes
Copy link
Contributor

@beer-1 beer-1 left a comment

Choose a reason for hiding this comment

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

requested some changes on

  • env
  • challenger
  • bridge executor's failed tx handling

bots/.envrc_sample Outdated Show resolved Hide resolved
bots/src/lib/query.ts Outdated Show resolved Hide resolved
bots/src/lib/query.ts Outdated Show resolved Hide resolved
bots/.envrc_sample Outdated Show resolved Hide resolved
bots/src/lib/storage.ts Outdated Show resolved Hide resolved
bots/src/worker/challenger/challenger.ts Outdated Show resolved Hide resolved
bots/src/worker/challenger/challenger.ts Outdated Show resolved Hide resolved
bots/src/worker/challenger/challenger.ts Outdated Show resolved Hide resolved
bots/src/worker/challenger/challenger.ts Outdated Show resolved Hide resolved
bots/src/worker/challenger/challenger.ts Outdated Show resolved Hide resolved
Vritra4
Vritra4 reviewed Dec 19, 2023
View reviewed changes
bots/src/lib/logger.ts Outdated Show resolved Hide resolved
@JSHan94 JSHan94 requested a review from a team as a code owner January 3, 2024 04:14
@JSHan94 JSHan94 requested a review from beer-1 January 18, 2024 09:45
@JSHan94 JSHan94 merged commit 8b6137e into main Jan 23, 2024
1 check passed
@beer-1 beer-1 deleted the feat/sdk branch April 17, 2024 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joon9823 joon9823 joon9823 left review comments

@Vritra4 Vritra4 Vritra4 left review comments

@beer-1 beer-1 Awaiting requested review from beer-1

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@JSHan94 @Vritra4 @joon9823 @beer-1