add the nft processing queue name in configs

multiversx / mx-api-service

Unified REST API facade for providing data related to the MultiversX blockchain

https://api.multiversx.com

GNU General Public License v3.0

30 stars 48 forks source link

add the nft processing queue name in configs #1251

Closed cfaur09 closed 1 month ago

cfaur09 commented 3 months ago

Reasoning

At the moment the queue: 'api-process-nfts' is hardcoded in the code and the same rabbitmq cannot be used by 2 different api setups

Proposed Changes

Add the queue name for nft process queue in configs

IosifGabriel commented 3 months ago

Hello, I still think there needs to be done some concatetation for the deadLetterExchange as it remained hardcoded

https://github.com/multiversx/mx-api-service/blob/5b0d9f1a7a976cb4c4e44116da8c1572d6337ac6/src/main.ts#L129 https://github.com/multiversx/mx-api-service/blob/5b0d9f1a7a976cb4c4e44116da8c1572d6337ac6/src/queue.worker/nft.worker/nft.worker.module.ts#L35

cfaur09 commented 3 months ago

Hello, I still think there needs to be done some concatetation for the deadLetterExchange as it remained hardcoded

https://github.com/multiversx/mx-api-service/blob/5b0d9f1a7a976cb4c4e44116da8c1572d6337ac6/src/main.ts#L129

https://github.com/multiversx/mx-api-service/blob/5b0d9f1a7a976cb4c4e44116da8c1572d6337ac6/src/queue.worker/nft.worker/nft.worker.module.ts#L35

Solved 🟢