Skip to content

Conversation

@alexjba
Copy link
Contributor

@alexjba alexjba commented Nov 19, 2025

What does the PR do

Closes #19305

Disabling IOS swaps for this release.
The wallet tab was already hooked to the feature flags. The only missing piece was the market tab.

Affected areas

IOS - swap

Architecture compliance

How to test

Login and try to find the Swap buttons.

Risk

low

@alexjba alexjba requested review from a team, caybro, micieslak and noeliaSD as code owners November 19, 2025 08:59
@alexjba
Copy link
Contributor Author

alexjba commented Nov 19, 2025

@sunleos The swaps will be disabled for 2.36 release branch only. Please let me know if we need to permanently disable it on IOS (master branch). I hope it's not needed 😄

@status-im-auto
Copy link
Member

status-im-auto commented Nov 19, 2025

Jenkins Builds

Click to see older builds (64)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ d6c8495 #1 2025-11-19 09:07:19 ~7 min tests/nim 📄log
✔️ d6c8495 #1 2025-11-19 09:10:09 ~10 min android/arm64 🤖apk 📲
✔️ d6c8495 #1 2025-11-19 09:15:08 ~15 min tests/ui 📄log
✔️ d6c8495 #1 2025-11-19 09:17:23 ~17 min ios/aarch64 📱ipa
✔️ d6c8495 #1 2025-11-19 09:17:30 ~17 min linux/x86_64 📦tgz
✔️ d6c8495 #1 2025-11-19 09:18:14 ~18 min macos/aarch64 🍎dmg
✔️ d6c8495 #1 2025-11-19 09:21:00 ~20 min macos/aarch64-nwaku 🍎dmg
✔️ d6c8495 #1 2025-11-19 09:23:57 ~23 min linux/x86_64-nwaku 📦tgz
✔️ d6c8495 #1 2025-11-19 09:26:09 ~25 min windows/x86_64 💿exe
✖️ d6c8495 pr19306 2025-11-19 09:35:10 ~17 min tests/e2e 📊rpt
✖️ d6c8495 PR19306 2025-11-19 09:52:34 ~26 min tests/e2e-windows 📊rpt
✔️ ac40c67 #2 2025-11-19 13:25:30 ~8 min tests/nim 📄log
✔️ ac40c67 #2 2025-11-19 13:27:27 ~10 min android/arm64 🤖apk 📲
✔️ ac40c67 #2 2025-11-19 13:30:09 ~12 min macos/aarch64 🍎dmg
✔️ ac40c67 #2 2025-11-19 13:31:57 ~14 min ios/aarch64 📱ipa
✔️ ac40c67 #2 2025-11-19 13:32:38 ~15 min tests/ui 📄log
✔️ ac40c67 #2 2025-11-19 13:33:01 ~15 min macos/aarch64-nwaku 🍎dmg
✔️ ac40c67 #2 2025-11-19 13:33:11 ~15 min linux/x86_64 📦tgz
✔️ ac40c67 #2 2025-11-19 13:33:59 ~16 min linux/x86_64-nwaku 📦tgz
✔️ ac40c67 #2 2025-11-19 13:43:25 ~25 min windows/x86_64 💿exe
✔️ ac40c67 pr19306 2025-11-19 13:51:17 ~18 min tests/e2e 📊rpt
✖️ ac40c67 PR19306 2025-11-19 14:09:55 ~26 min tests/e2e-windows 📊rpt
✔️ 5204a53 #3 2025-11-19 15:17:26 ~6 min tests/nim 📄log
✔️ 5204a53 #3 2025-11-19 15:21:21 ~10 min android/arm64 🤖apk 📲
✔️ 5204a53 #3 2025-11-19 15:23:14 ~12 min macos/aarch64 🍎dmg
✔️ 5204a53 #3 2025-11-19 15:24:22 ~13 min macos/aarch64-nwaku 🍎dmg
✔️ 5204a53 #3 2025-11-19 15:24:49 ~13 min ios/aarch64 📱ipa
✔️ 5204a53 #3 2025-11-19 15:26:27 ~15 min linux/x86_64-nwaku 📦tgz
✔️ 5204a53 #3 2025-11-19 15:27:49 ~16 min linux/x86_64 📦tgz
✔️ 5204a53 #3 2025-11-19 15:28:54 ~17 min tests/ui 📄log
✔️ 5204a53 #3 2025-11-19 15:40:12 ~29 min windows/x86_64 💿exe
✔️ 5204a53 pr19306 2025-11-19 15:43:36 ~15 min tests/e2e 📊rpt
✖️ 5204a53 PR19306 2025-11-19 16:08:41 ~28 min tests/e2e-windows 📊rpt
✔️ 98157a58 #4 2025-11-24 17:35:29 ~19 min android/arm64 🤖apk 📲
✔️ 1c619ea4 #5 2025-11-25 17:45:40 ~29 min android/arm64 🤖apk 📲
✔️ c257272e #6 2025-11-26 17:38:22 ~22 min android/arm64 🤖apk 📲
✔️ 577c13d7 #7 2025-11-27 17:37:00 ~21 min android/arm64 🤖apk 📲
✔️ 4625a76d #8 2025-11-28 17:27:12 ~11 min android/arm64 🤖apk 📲
49c6a0a #4 2025-11-29 02:24:06 ~10 min macos/aarch64 📄log
✔️ 49c6a0a #9 2025-11-29 02:24:17 ~10 min android/arm64 🤖apk 📲
49c6a0a #4 2025-11-29 02:27:42 ~14 min macos/aarch64-nwaku 📄log
✔️ 49c6a0a #4 2025-11-29 02:28:35 ~15 min tests/nim 📄log
✔️ 49c6a0a #4 2025-11-29 02:35:09 ~21 min tests/ui 📄log
✔️ 49c6a0a #4 2025-11-29 02:35:43 ~22 min linux/x86_64 📦tgz
✔️ 49c6a0a #4 2025-11-29 02:35:46 ~22 min ios/aarch64 📱ipa
✔️ 49c6a0a #4 2025-11-29 02:38:53 ~25 min linux/x86_64-nwaku 📦tgz
✔️ 49c6a0a #4 2025-11-29 02:44:57 ~31 min windows/x86_64 💿exe
✔️ 49c6a0a pr19306 2025-11-29 02:50:54 ~15 min tests/e2e 📊rpt
✖️ 49c6a0a PR19306 2025-11-29 03:13:39 ~28 min tests/e2e-windows 📊rpt
✔️ 9fe0ff93 #10 2025-11-29 17:27:55 ~12 min android/arm64 🤖apk 📲
✔️ bd83fa6a #11 2025-12-01 17:38:31 ~22 min android/arm64 🤖apk 📲
✔️ 88d87d65 #12 2025-12-02 17:44:23 ~28 min android/arm64 🤖apk 📲
a1794b6 #5 2025-12-02 20:10:37 ~5 min macos/aarch64 📄log
✔️ a1794b6 #5 2025-12-02 20:12:43 ~7 min tests/nim 📄log
✔️ a1794b6 #13 2025-12-02 20:15:24 ~10 min android/arm64 🤖apk 📲
✔️ a1794b6 #5 2025-12-02 20:19:05 ~14 min tests/ui 📄log
✔️ a1794b6 #5 2025-12-02 20:23:06 ~18 min linux/x86_64 📦tgz
✔️ a1794b6 #5 2025-12-02 20:25:27 ~20 min ios/aarch64 📱ipa
✔️ a1794b6 #5 2025-12-02 20:28:42 ~23 min linux/x86_64-nwaku 📦tgz
✔️ a1794b6 #5 2025-12-02 20:29:37 ~24 min macos/aarch64-nwaku 🍎dmg
✔️ a1794b6 #5 2025-12-02 20:31:19 ~26 min windows/x86_64 💿exe
✔️ a1794b6 pr19306 2025-12-02 20:39:51 ~16 min tests/e2e 📊rpt
✖️ a1794b6 PR19306 2025-12-02 20:54:09 ~22 min tests/e2e-windows 📊rpt
✔️ 01a26654 #14 2025-12-03 17:37:12 ~21 min android/arm64 🤖apk 📲
Commit #️⃣ Finished (UTC) Duration Platform Result
ee3ece6 #6 2025-12-03 18:09:09 ~6 min macos/aarch64 📄log
✔️ ee3ece6 #6 2025-12-03 18:10:10 ~7 min tests/nim 📄log
✔️ ee3ece6 #15 2025-12-03 18:13:10 ~10 min android/arm64 🤖apk 📲
✔️ ee3ece6 #6 2025-12-03 18:15:37 ~13 min tests/ui 📄log
✖️ ee3ece6 #6 2025-12-03 18:15:47 ~13 min ios/aarch64 📱ipa
✔️ ee3ece6 #6 2025-12-03 18:19:35 ~17 min linux/x86_64 📦tgz
✔️ ee3ece6 #6 2025-12-03 18:23:40 ~21 min linux/x86_64-nwaku 📦tgz
✔️ ee3ece6 #6 2025-12-03 18:25:44 ~23 min macos/aarch64-nwaku 🍎dmg
✔️ ee3ece6 #6 2025-12-03 18:28:59 ~26 min windows/x86_64 💿exe
✖️ ee3ece6 pr19306 2025-12-03 18:34:12 ~14 min tests/e2e 📊rpt
✖️ ee3ece6 PR19306 2025-12-03 18:54:07 ~24 min tests/e2e-windows 📊rpt
✔️ 69749ef #16 2025-12-04 17:09:59 ~10 min android/arm64 🤖apk 📲
✖️ 69749ef #7 2025-12-04 17:10:07 ~10 min ios/aarch64 📱ipa
✔️ 69749ef #7 2025-12-04 17:10:13 ~10 min tests/nim 📄log
✔️ 69749ef #7 2025-12-04 17:15:11 ~15 min linux/x86_64-nwaku 📦tgz
✔️ 69749ef #7 2025-12-04 17:16:08 ~16 min macos/aarch64-nwaku 🍎dmg
✔️ 69749ef #7 2025-12-04 17:16:15 ~16 min macos/aarch64 🍎dmg
✔️ 69749ef #7 2025-12-04 17:18:57 ~19 min linux/x86_64 📦tgz
✔️ 69749ef #7 2025-12-04 17:19:04 ~19 min tests/ui 📄log
✔️ 69749ef #7 2025-12-04 17:29:27 ~29 min windows/x86_64 💿exe
✖️ 69749ef PR19306 2025-12-04 17:48:04 ~18 min tests/e2e-windows 📊rpt
69749ef pr19306 2025-12-04 17:51:44 ~32 min tests/e2e 📄log

@sunleos
Copy link
Collaborator

sunleos commented Nov 19, 2025

@sunleos The swaps will be disabled for 2.36 release branch only. Please let me know if we need to permanently disable it on IOS (master branch). I hope it's not needed 😄

Yes, it is OK to have it disabled just for 2.36 for now.

@noeliaSD
Copy link
Contributor

@alexjba we want this pr directly to master. @alaibe will cut it again today once everything needed is on master

@alexjba
Copy link
Contributor Author

alexjba commented Nov 19, 2025

@alexjba we want this pr directly to master. @alaibe will cut it again today once everything needed is on master

then maybe we can wait for the next cut? Doesn't really makes sense me to have it in master just to revert it after the cut.

Copy link
Contributor

@noeliaSD noeliaSD left a comment

Choose a reason for hiding this comment

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

LGTM!

  • Tested on ios simulator.
  • Tested on desktop.

@noeliaSD
Copy link
Contributor

@alexjba we want this pr directly to master. @alaibe will cut it again today once everything needed is on master

then maybe we can wait for the next cut? Doesn't really makes sense me to have it in master just to revert it after the cut.

Got it! I was thinking we wanted to keep this disabled state in master too. If it's just something for the release, then it points to the correct branch already! Thanks for clarification!

Copy link
Member

@micieslak micieslak left a comment

Choose a reason for hiding this comment

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

Tested, looks all good.

@caybro
Copy link
Member

caybro commented Nov 19, 2025

We still need this in master; everything except the physical setting of the flag for iOS (d:swap_disabled)

@jrainville jrainville force-pushed the fix/disable-swap-ios branch from ee3ece6 to 69749ef Compare December 4, 2025 16:59
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.

6 participants