-
Notifications
You must be signed in to change notification settings - Fork 3
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: Migrate to v3 part #887
Conversation
ec398a9
to
1e848e4
Compare
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 found some strange displaying of activities here: https://pr-887-aescan.stg.service.aepps.com/accounts/ak_fURxL4TQ3jSGvhpEzZeS7szmmNbiMwtDnUtfwRf3AqqPexXyK?type=activities
is this correct displaying? https://pr-887-aescan-testnet.stg.service.aepps.com/tokens/ct_Sn3iTuEXniEAAiojffR5nmigCffQZwy3sNVHG6Zxof8HeNXXv?type=events
@janmichek could you check it please?
I was checking this. It's even problem in production, so not related to the change itself. I don't see any suspicious on that. Also looks the same as on production. What exactly you think its the issue?
I think it's ok. Same as production. What exactly you think its the issue? |
In ad 2 there was some extra data - address in data field and amount under empty field. But now it looks good. ad 3. Not the issue rather the view that amount, for ex 15 MDT is placing under "empty" column |
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.
the rest is LGTM
Thanks will be released in the second release |
Description
another part of migration endoints
when testing, focus on account tokens listing and token events counter, nft detail
partly resolves #690
Demo
Should work without any change
Checklist: