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

Seems newest list from landing is gone #2747

Closed
yangwao opened this issue Apr 2, 2022 · 11 comments · Fixed by #2749
Closed

Seems newest list from landing is gone #2747

yangwao opened this issue Apr 2, 2022 · 11 comments · Fixed by #2749
Labels
$ ~<50usd A-landing issues related to landing page bug Something isn't working p3 non-core, affecting less than 40%

Comments

@yangwao
Copy link
Member

yangwao commented Apr 2, 2022

Uncaught (in promise) TypeError: Cannot read properties of null (reading 'id')
    at 41d068c.js:1:10873
    at Array.map (<anonymous>)
    at f.<anonymous> (41d068c.js:1:10838)
    at v (152af89.js:2:16194)
    at Generator._invoke (152af89.js:2:15982)
    at Generator.next (152af89.js:2:16623)
    at r (152af89.js:2:2747)
    at l (152af89.js:2:2950)
    at 152af89.js:2:3009
    at new Promise (<anonymous>)

image

@yangwao yangwao added bug Something isn't working $ ~<50usd p1 preventing everyone from using app A-landing issues related to landing page labels Apr 2, 2022
@Jarsen136
Copy link
Contributor

I will check it

@preschian
Copy link
Member

strange, seems fixed for now? Does something change from the server-side?

@Jarsen136
Copy link
Contributor

Jarsen136 commented Apr 2, 2022

It seems work well now. I think that maybe back-end interface was not stable at that time.

@yangwao
Copy link
Member Author

yangwao commented Apr 2, 2022

Some nft from that list wasn't properly handled I bet https://kodadot.xyz/rmrk/gallery?search=&sort=UPDATED_AT_DESC&page=1

naturally people are listing new nft so hard to see what was making issue :)

@Jarsen136
Copy link
Contributor

Oh I see. I will try to reproduce it.

@yangwao yangwao added p3 non-core, affecting less than 40% and removed p1 preventing everyone from using app labels Apr 2, 2022
@yangwao
Copy link
Member Author

yangwao commented Apr 2, 2022

Yeah hence it isn't occurring lowering priority, if we don't solve it happy to close it and snipping if it happen again lol 😅

@Jarsen136
Copy link
Contributor

Yeah hence it isn't occurring lowering priority, if we don't solve it happy to close it and snipping if it happen again lol 😅

Don't worry. I have caught the error. Give me some time to fix it 🚶

image

@preschian
Copy link
Member

woaah, the meta = null

@Jarsen136 Jarsen136 mentioned this issue Apr 2, 2022
17 tasks
@yangwao
Copy link
Member Author

yangwao commented Apr 2, 2022

Yeah hence it isn't occurring lowering priority, if we don't solve it happy to close it and snipping if it happen again lol 😅

Don't worry. I have caught the error. Give me some time to fix it 🚶

image

Seems you're a new superior bug hunter 🤟🤘 engineering

@XyloDrone
Copy link

Hey sers, what's the deal with this issue? Seems fixed?

@Jarsen136
Copy link
Contributor

Hey sers, what's the deal with this issue? Seems fixed?

It's an occasional problem that only occurs when certain nfts are listed.

already fixed in #2749

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
$ ~<50usd A-landing issues related to landing page bug Something isn't working p3 non-core, affecting less than 40%
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants