Skip to content

Conversation

dt-iohk
Copy link
Contributor

@dt-iohk dt-iohk commented Sep 4, 2025

Description

See input-output-hk/catalyst-voices#3370 for description.

Related Issue(s)

Refers input-output-hk/catalyst-voices#3266

Please confirm the following checks

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream module

@dt-iohk dt-iohk self-assigned this Sep 4, 2025
@dt-iohk dt-iohk added this to Catalyst Sep 4, 2025
@dt-iohk dt-iohk added do not merge yet PR is not ready to be merged yet do not review yet Do not review yet labels Sep 4, 2025
@dt-iohk dt-iohk added squad: face Frontend Catalyst Experience Team no-track Used to skip tracking in Swarmia analytics, etc. labels Sep 4, 2025
@dt-iohk dt-iohk moved this from New to 🏗 In progress in Catalyst Sep 4, 2025
@dt-iohk dt-iohk added review me PR is ready for review and removed do not merge yet PR is not ready to be merged yet do not review yet Do not review yet no-track Used to skip tracking in Swarmia analytics, etc. labels Sep 11, 2025
@dt-iohk dt-iohk moved this from 🏗 In progress to 👀 In review in Catalyst Sep 11, 2025
@dt-iohk dt-iohk marked this pull request as ready for review September 11, 2025 12:40
Copy link
Contributor

@damian-molinski damian-molinski left a comment

Choose a reason for hiding this comment

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

LGTM

# https://blog.rust-lang.org/2025/04/04/c-abi-changes-for-wasm32-unknown-unknown/
RUN flutter_rust_bridge_codegen build-web
# TODO(dt-iohk): revert to official version when changes are merged
GIT CLONE --branch feat/wasm_bindgen_configurable https://github.com/dt-iohk/flutter_rust_bridge.git /usr/local/flutter_rust_bridge
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm okay of doing it this way, but let's comment out flutter_rust_bridge thing from INSTALL_TOOLS and tool-flutter-rust-bridge-codegen and make some note there.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. These changes are temporary. I hope it's short lived rather than long-lived but depends on how much time it'll take to merge it to the original lib.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For clarification, I'm not going to merge this PR while it still points to my own fork, I'll wait until we have the IOHK one.

Copy link
Contributor

Choose a reason for hiding this comment

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

Besides from that LGTM

@dt-iohk dt-iohk moved this from 👀 In review to 🛑 Blocked in Catalyst Sep 16, 2025
@dt-iohk dt-iohk added draft Draft and removed review me PR is ready for review labels Sep 16, 2025
@dt-iohk dt-iohk marked this pull request as draft September 16, 2025 21:20
@dt-iohk dt-iohk added do not merge yet PR is not ready to be merged yet do not review yet Do not review yet no-track Used to skip tracking in Swarmia analytics, etc. labels Sep 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge yet PR is not ready to be merged yet do not review yet Do not review yet draft Draft no-track Used to skip tracking in Swarmia analytics, etc. squad: face Frontend Catalyst Experience Team
Projects
Status: 🛑 Blocked
Development

Successfully merging this pull request may close these issues.

3 participants