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

ArgumentError: comparison with nil is forbidden as it is unsafe. If you want to check if a value is nil, use is_... #1237

Closed
sentry-io bot opened this issue Dec 25, 2022 · 2 comments · Fixed by #1238 or Magickbase/godwoken-explorer-ui#816
Assignees
Labels
bug Something isn't working

Comments

@sentry-io
Copy link

sentry-io bot commented Dec 25, 2022

Sentry Issue: MAINNET-GWSCAN-9

ArgumentError: comparison with nil is forbidden as it is unsafe. If you want to check if a value is nil, use is_nil/1 instead
  File "lib/ecto/query/builder.ex", line 958, in Ecto.Query.Builder.not_nil!/1
  File "lib/godwoken_explorer/udt.ex", line 239, in GodwokenExplorer.UDT.minted_count/1
  File "lib/godwoken_explorer/graphql/resolovers/udt.ex", line 53, in GodwokenExplorer.Graphql.Resolvers.UDT.minted_count/3
  File "lib/absinthe/resolution.ex", line 209, in Absinthe.Resolution.call/2
  File "lib/absinthe/phase/document/execution/resolution.ex", line 230, in Absinthe.Phase.Document.Execution.Resolution.reduce_resolution/1
...
(3 additional frame(s) were not displayed)

(ArgumentError comparison with nil is forbidden as it is unsafe. If you want to check if a value is nil, use is_nil/1 instead)
@zmcNotafraid zmcNotafraid assigned zmcNotafraid and unassigned Naupio Dec 26, 2022
@zmcNotafraid zmcNotafraid added the bug Something isn't working label Dec 26, 2022
@zmcNotafraid zmcNotafraid added this to the 2022/12/21 - 2022-12/28 milestone Dec 26, 2022
@zmcNotafraid zmcNotafraid moved this to In Progress in Nervos Wallet/Explorer Dec 26, 2022
@zmcNotafraid zmcNotafraid assigned qiweiii and unassigned zmcNotafraid Dec 26, 2022
@zmcNotafraid zmcNotafraid moved this from In Progress to Todo in Nervos Wallet/Explorer Dec 26, 2022
@zmcNotafraid
Copy link
Member

Frontend will remove this column's use

Repository owner moved this from Todo to Production in Nervos Wallet/Explorer Dec 26, 2022
@Keith-CY
Copy link
Member

It was introduced by using minted_count field on an ERC 20 page, but only ERC 721/1155 has a minted_count field.

The erroneous code is at https://github.com/Magickbase/godwoken-explorer-ui/blob/cd0ffb642400e8f9138b8d070f1b0bc9f2f349df/pages/token/%5Bid%5D.tsx#L80

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Archived in project
4 participants