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.

Sign up for GitHub

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

Jump to bottom

a bug in hamt traversal example #122

Closed
lidel opened this issue Jul 13, 2023 · 2 comments · Fixed by ipfs/ipld-explorer-components#384 or #123
Closed

a bug in hamt traversal example #122

lidel opened this issue Jul 13, 2023 · 2 comments · Fixed by ipfs/ipld-explorer-components#384 or #123
Assignees
@SgtPooki
Labels
effort/hours Estimated to take one or several hours exp/intermediate Prior experience is likely helpful kind/bug A bug in existing code (including security flaws)

Comments

@lidel
Copy link
Collaborator

lidel commented Jul 13, 2023 •
edited
Loading

To Reproduce:

  1. Open https://explore.ipld.io/#/explore/bafybeiaysi4s6lnjev27ln5icwm6tueaw2vdykrtjkwiphwekaywqhcjze/1D_exceptions
  2. Click on 2023-07-13_19-25_1
    7 0CA%2fMonitor_%2f%2f → nothing changes, same node is shown, header shows broken(?) breadcrumb:
    2023-07-13_19-25 and the same node is shown
  3. Click on 2023-07-13_19-25_1 again → nothing happens

Expected:
Click on 2023-07-13_19-25_1 should step into its CID
@lidel lidel added the need/triage Needs initial labeling and prioritization label Jul 13, 2023
@github-project-automation github-project-automation bot moved this to Needs Grooming in IPFS-GUI (PL EngRes) Jul 14, 2023
@SgtPooki SgtPooki added kind/bug A bug in existing code (including security flaws) exp/intermediate Prior experience is likely helpful effort/hours Estimated to take one or several hours and removed need/triage Needs initial labeling and prioritization labels Jul 17, 2023
@SgtPooki SgtPooki moved this from Needs Grooming to Planned / Backlog in IPFS-GUI (PL EngRes) Jul 17, 2023
@SgtPooki
Copy link
Contributor

Thanks @lidel , I was able to reproduce and put this in our planned backlog of items

@SgtPooki SgtPooki self-assigned this Jul 24, 2023
@SgtPooki SgtPooki mentioned this issue Jul 24, 2023
Merged
SgtPooki added a commit to ipfs/ipld-explorer-components that referenced this issue Jul 24, 2023
@SgtPooki
fix: url-encoded dag-PB names can be explored
ba6a09a 
fixes ipld/explore.ipld.io#122
SgtPooki added a commit to ipfs/ipld-explorer-components that referenced this issue Jul 24, 2023
@SgtPooki
fix: url-encoded dag-PB names can be explored (#384)
d5c6899 
fixes ipld/explore.ipld.io#122
@github-project-automation github-project-automation bot moved this from Planned / Backlog to Done in IPFS-GUI (PL EngRes) Jul 24, 2023
@SgtPooki SgtPooki reopened this Jul 24, 2023
@github-project-automation github-project-automation bot moved this from Done to Needs Grooming in IPFS-GUI (PL EngRes) Jul 24, 2023
@SgtPooki
Copy link
Contributor

re-opening cause we have to deploy the new explorer components here.

This was referenced Jul 24, 2023
Merged
Merged
@github-project-automation github-project-automation bot moved this from Needs Grooming to Done in IPFS-GUI (PL EngRes) Jul 27, 2023
@lidel lidel mentioned this issue Sep 5, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SgtPooki SgtPooki

Labels
effort/hours Estimated to take one or several hours exp/intermediate Prior experience is likely helpful kind/bug A bug in existing code (including security flaws)
Projects
None yet
Milestone
No milestone
2 participants
@lidel @SgtPooki