Skip to content

Comments

feat: deprecate tree_path and rlnDB#3577

Merged
darshankabariya merged 7 commits intomasterfrom
tree_path_depr
Sep 26, 2025
Merged

feat: deprecate tree_path and rlnDB#3577
darshankabariya merged 7 commits intomasterfrom
tree_path_depr

Conversation

@darshankabariya
Copy link
Contributor

@darshankabariya darshankabariya commented Sep 22, 2025

Description

After post-sync, the merkle tree is maintained by the smart contract, so treepath is no longer a valid parameter. That’s why we are deprecating it from the config level down to the database level. will effect rlnTreeConfig, treePath, Inspect tool and rlnDB.

status-im/infra-role-nim-waku#47
logos-messaging/docs.waku.org#248
https://github.com/status-im/infra-misc/pull/432
logos-messaging/logos-delivery-simulator#112

Issue

@github-actions
Copy link

github-actions bot commented Sep 22, 2025

You can find the image built from this PR at

quay.io/wakuorg/nwaku-pr:3577

Built from feea395

@github-actions
Copy link

This PR may contain changes to configuration options of one of the apps.

If you are introducing a breaking change (i.e. the set of options in latest release would no longer be applicable) make sure the original option is preserved with a deprecation note for 2 following releases before it is actually removed.

Please also make sure the label release-notes is added to make sure any changes to the user interface are properly announced in changelog and release notes.

@darshankabariya darshankabariya marked this pull request as ready for review September 23, 2025 21:13
@darshankabariya darshankabariya changed the title feat: deprecate tree_path and remove userMessageLimit feat: deprecate tree_path and rlnDB Sep 24, 2025
Copy link
Collaborator

@Ivansete-status Ivansete-status left a comment

Choose a reason for hiding this comment

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

Copy link
Collaborator

@fryorcraken fryorcraken left a comment

Choose a reason for hiding this comment

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

excellent

@darshankabariya darshankabariya merged commit 5118fc3 into master Sep 26, 2025
10 of 12 checks passed
@darshankabariya darshankabariya deleted the tree_path_depr branch September 26, 2025 09:17
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