Skip to content

fix: remove replica node url from submitter deploys#985

Merged
norswap merged 4 commits intomasterfrom
ag/rm-replica
Jul 23, 2025
Merged

fix: remove replica node url from submitter deploys#985
norswap merged 4 commits intomasterfrom
ag/rm-replica

Conversation

@aodhgan
Copy link
Contributor

@aodhgan aodhgan commented Jul 9, 2025

The replica is so far behind tip/head of chain, it causes all sorts of issues on submitter re-boot. This fixes it for now by only using the caldera node.

Linked Issues

  • closes #XXX
  • closes HAPPY-XXX

Description

< DESCRIPTION GOES HERE >

  • Include all relevant context (but no need to repeat the issue's content).
  • Draw attention to new, noteworthy & unintuitive elements.
Toggle Checklist

Checklist

Basics

  • B1. I have applied the proper label & proper branch name (e.g. norswap/build-system-caching).
  • B2. This PR is not so big that it should be split & addresses only one concern.
  • B3. The PR targets the lowest branch it can (ideally master).

Reminder: PR review guidelines

Correctness

  • C1. Builds and passes tests.
  • C2. The code is properly parameterized & compatible with different environments (e.g. local,
    testnet, mainnet, standalone wallet, ...).
  • C3. I have manually tested my changes & connected features.

< INDICATE BROWSER, DEMO APP & OTHER ENV DETAILS USED FOR TESTING HERE >

< INDICATE TESTED SCENARIOS (USER INTERFACE INTERACTION, CODE FLOWS) HERE >

  • C4. I have performed a thorough self-review of my code after submitting the PR,
    and have updated the code & comments accordingly.

Architecture & Documentation

  • D1. I made it easy to reason locally about the code, by (1) using proper abstraction boundaries,
    (2) commenting these boundaries correctly, (3) adding inline comments for context when needed.
  • D2. All public-facing APIs are documented in the docs.
    Public APIS and meaningful (non-local) internal APIs are properly documented in code comments.
  • D3. If appropriate, the general architecture of the code is documented in a code comment or
    in a Markdown document.
  • D4. An appropriate Changeset has been generated (and committed) with make changeset for
    breaking and meaningful changes in packages (not required for cleanups & refactors).

Copy link
Collaborator

norswap commented Jul 20, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jul 20, 2025

Deploying happychain with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6549a09
Status: ✅  Deploy successful!
Preview URL: https://445254a7.happychain.pages.dev
Branch Preview URL: https://ag-rm-replica.happychain.pages.dev

View logs

@norswap norswap force-pushed the ag/rm-replica branch 2 times, most recently from 4143b01 to 6549a09 Compare July 21, 2025 16:11
@norswap norswap merged commit d44896a into master Jul 23, 2025
2 checks passed
@norswap norswap deleted the ag/rm-replica branch July 23, 2025 10:28
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.

3 participants