Skip to content

[TEMP] Simulate offline behavior by using dummy config values (offline state) #17720

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

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

noeliaSD
Copy link
Contributor

@noeliaSD noeliaSD commented Apr 2, 2025

⚠️ This PR won't be merged ⚠️

This PR has been created ONLY to make some explorations about how the app will behave (mostly the wallet) when third-parties providers are disabled, so it can be used as a preliminar test for the team for the new "private mode" it's under discussion.

@noeliaSD noeliaSD requested a review from dlipicar April 2, 2025 11:12
@noeliaSD noeliaSD self-assigned this Apr 2, 2025
@status-im-auto
Copy link
Member

status-im-auto commented Apr 2, 2025

Jenkins Builds

Click to see older builds (7)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ f1ab2c8 #1 2025-04-02 11:21:33 ~8 min macos/aarch64 🍎dmg
✔️ f1ab2c8 #1 2025-04-02 11:23:37 ~10 min tests/nim 📄log
✔️ f1ab2c8 #1 2025-04-02 11:26:24 ~13 min tests/ui 📄log
✔️ f1ab2c8 #1 2025-04-02 11:28:13 ~15 min macos/x86_64 🍎dmg
✔️ f1ab2c8 #1 2025-04-02 11:35:50 ~22 min linux-nix/x86_64 📦tgz
✔️ f1ab2c8 #1 2025-04-02 11:36:35 ~23 min windows/x86_64 💿exe
✔️ f1ab2c8 #1 2025-04-02 11:38:58 ~26 min linux/x86_64 📦tgz
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ ee37182 #2 2025-04-02 12:47:17 ~8 min tests/nim 📄log
✔️ ee37182 #2 2025-04-02 12:47:19 ~8 min macos/aarch64 🍎dmg
✔️ ee37182 #2 2025-04-02 12:51:21 ~12 min macos/x86_64 🍎dmg
✔️ ee37182 #2 2025-04-02 12:52:08 ~13 min tests/ui 📄log
✔️ ee37182 #2 2025-04-02 13:00:15 ~21 min windows/x86_64 💿exe
✔️ ee37182 #2 2025-04-02 13:03:54 ~25 min linux/x86_64 📦tgz
✔️ ee37182 #2 2025-04-02 13:05:00 ~26 min linux-nix/x86_64 📦tgz
✔️ 2c253a1 #3 2025-04-02 13:22:02 ~7 min macos/aarch64 🍎dmg
✔️ 2c253a1 #3 2025-04-02 13:23:46 ~8 min tests/nim 📄log
✔️ 2c253a1 #3 2025-04-02 13:28:05 ~13 min tests/ui 📄log
✔️ 2c253a1 #3 2025-04-02 13:29:22 ~14 min macos/x86_64 🍎dmg
✔️ 2c253a1 #3 2025-04-02 13:35:17 ~20 min windows/x86_64 💿exe
✔️ 2c253a1 #3 2025-04-02 13:35:56 ~21 min linux-nix/x86_64 📦tgz
✔️ 2c253a1 #3 2025-04-02 13:41:00 ~26 min linux/x86_64 📦tgz

@dlipicar
Copy link
Contributor

dlipicar commented Apr 2, 2025

Hmm I don't think this is enough:

  • Market providers (coingecko, cryptocompare) are accessed without an API key
  • If I remember correctly, Alchemy will try to use the public endpoints (no API key) if there's no keys or the injected keys fail.
  • Proxy credentials also need to be replaced

@dlipicar
Copy link
Contributor

dlipicar commented Apr 2, 2025

I'd suggest using a firewall/hosts file to block all these
image
and starting with a fresh account so no cache is available. Otherwise we need to touch status-go.

@noeliaSD noeliaSD force-pushed the test/check-offline-behavior branch from f1ab2c8 to ee37182 Compare April 2, 2025 12:38
@dlipicar
Copy link
Contributor

dlipicar commented Apr 2, 2025

Oh wait I'm also missing onramp, bridge and swap providers

@dlipicar
Copy link
Contributor

dlipicar commented Apr 2, 2025

Bumped status-go to get status-im/status-go#6499
That should do the trick

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants