-
Notifications
You must be signed in to change notification settings - Fork 45
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
add the nft processing queue name in configs #1251
base: development
Are you sure you want to change the base?
add the nft processing queue name in configs #1251
Conversation
Hello, I still think there needs to be done some concatetation for the deadLetterExchange as it remained hardcoded Line 129 in 5b0d9f1
|
Solved 🟢 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would make here a feature called "processNfts" and add there both the normal queue and the dead letter queue name. when reading the activation flag, try to read it first from features and then also from the "old" location to maintain backwards compatibility. update all config files with the new location. in this way we have all process nft-related functionality grouped together
Reasoning
Proposed Changes