Skip to content
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

Feat/update owner for each cw721 activity ( main) #349

Closed
wants to merge 11 commits into from

Conversation

phamphong9981
Copy link
Collaborator

No description provided.

await this.updateOwnerForMintActs(
unprocessedActivities.filter((e) => e.action === CW721_ACTION.MINT)
);
const tokens = unprocessedActivities.reduce(
Copy link
Member

Choose a reason for hiding this comment

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

code lại

@phamphong9981 phamphong9981 changed the title fix: cw721 activity save owner at each time ( main ) Feat/update owner for each cw721 activity ( main) Aug 31, 2023

export async function up(knex: Knex): Promise<void> {
await knex.schema.alterTable('cw721_activity', (table) => {
table.string('owner');
Copy link
Member

Choose a reason for hiding this comment

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

để nguyên định nghĩa from với to, em thêm sender thì nó hợp lý hơn

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

from và to hiện tại đang có data, thực chất from đang là data của sender, nếu thêm cột sender thì phải chuyển data từ from qua ( hoặc rename cột ). Em thêm cột owner thì chỉ cần fill data vào mà em thấy cũng tạm chấp nhận được, cái tên cột mình hiểu là đc mà :))

Copy link
Member

Choose a reason for hiding this comment

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

😛

queueName: BULL_JOB_NAME.UPDATE_CW721_ACTIVITY_OWNER_BY_TOKEN,
jobName: BULL_JOB_NAME.UPDATE_CW721_ACTIVITY_OWNER_BY_TOKEN,
})
async updateCw721ActOwnerByToken(_payload: {
Copy link
Member

Choose a reason for hiding this comment

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

hiện tại trên xstaxy cũng chỉ có 25k activity, load hết ra mà sửa trong 1 migration
sau đó thì gán luôn lúc có activity mới

Copy link
Collaborator Author

@phamphong9981 phamphong9981 Sep 5, 2023

Choose a reason for hiding this comment

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

Trước kia em đã có nghĩ giải quyết theo hướng chạy migration update data cũ và sửa logic code. Nhưng việc sửa logic code sẽ gặp những vấn đề là:

  • code phình to
  • logic tương đối phức tạp
  • khó debug/test
  • có nhiều trường hợp khi một phần data nằm trong range [startBlock, endBlock] khi mình bắt được và một phần data nằm trong DB

Em muốn thống nhất là data khi trong DB rồi thì mình bốc ra xử lý những cái chưa update owner

@phamphong9981
Copy link
Collaborator Author

@peara em sửa lại theo migration + update code r anh nhé

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