Skip to content
This repository was archived by the owner on Aug 21, 2025. It is now read-only.

KIBA-1572: Create tbl_account_ens_names#443

Open
Femi-Ogunkola wants to merge 20 commits into
mainfrom
tbl_account_ens_names
Open

KIBA-1572: Create tbl_account_ens_names#443
Femi-Ogunkola wants to merge 20 commits into
mainfrom
tbl_account_ens_names

Conversation

@Femi-Ogunkola

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread api/notd/api/models_v1.py
Comment thread api/notd/ens_manager.py
Comment thread api/notd/manager.py Outdated
Comment thread api/notd/ens_manager.py Outdated
await self.saver.delete_account_ens_name(accountEnsNameId=accountEnsName.accountEnsNameId, connection=connection)
ensName = await self.accountEnsNameProcessor.get_ens_name(accountAddress=accountAddress)
accountEnsName = await self.saver.create_account_ens_name(accountAddress=accountAddress, ensName=ensName, connection=connection)
print(accountEnsName)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

remove

row = result.first()
if not row:
raise NotFoundException(message=f'AccountCollectionGm for accountAddress: {accountAddress} not found')
accountCollectionGm = account_ens_name_from_row(row)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

rename accountEnsName

Comment thread api/notd/ens_manager.py Outdated
if accountEnsName:
if accountEnsName.ensName and accountEnsName.updatedDate > date_util.datetime_from_now(days=-3):
return accountEnsName
await self.saver.delete_account_ens_name(accountEnsNameId=accountEnsName.accountEnsNameId, connection=connection)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Update don't delete

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant