Skip to content

Scan non-bounced address#12

Open
smman89 wants to merge 12 commits intotonbase:mainfrom
smman89:fix-check-none-bounce-address
Open

Scan non-bounced address#12
smman89 wants to merge 12 commits intotonbase:mainfrom
smman89:fix-check-none-bounce-address

Conversation

@smman89
Copy link
Copy Markdown

@smman89 smman89 commented Mar 30, 2024

Scan non-bounced address (example UQ...._Xsd)

Copy link
Copy Markdown
Collaborator

@highfeed highfeed left a comment

Choose a reason for hiding this comment

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

Необходима доработка.

Comment thread monitors/addresses.js
const updateAddresses = async () => {
try {
const { data } = await axios.get('https://catchain.github.io/tonscan/src/addrbook.json')
const { data } = await axios.get('https://address-book.tonscan.org/addresses.json')
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

А ты уверен, что оно будет работать? Формат может быть разный.

Comment thread services/transactionProcessor.js Outdated
const addresses = await addressRepository.getByAddress([transaction.from, transaction.to], {
const noneBouncedAddressFrom = getNoneBouncedAddress(transaction.from)
const noneBouncedAddressTo = getNoneBouncedAddress(transaction.to)
console.log(noneBouncedAddressFrom)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Дебаги нужно удалять.

Comment thread services/transactionProcessor.js Outdated
is_deleted: false,
'notifications.is_enabled': true,
$expr: { $gte: [transaction.nanoValue, '$notifications.min_amount'] },
$expr: { $gte: [transaction.nanoValue, '$notificatioLns.min_amount'] },
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Тут точно всё ок?

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.

2 participants