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

Breadcrumb ancestors deallocated #2797

Open
raineorshine opened this issue Jan 28, 2025 · 1 comment · Fixed by #2798
Open

Breadcrumb ancestors deallocated #2797

raineorshine opened this issue Jan 28, 2025 · 1 comment · Fixed by #2798
Labels
bug Something isn't working

Comments

@raineorshine
Copy link
Contributor

raineorshine commented Jan 28, 2025

Steps to Reproduce

  1. Set FREE_THOUGHTS_THRESHOLD and FREE_THOUGHTS_MARGIN to 0.
  2. Set FREE_THOUGHTS_THROTTLE to 10.
  3. Import Sample 1k thoughts.
  4. Create thought Power in the root context.
  5. Create thought Kshitigarbha in the root context.
  6. Run the following in the console, or in AppComponent on load:
const em = (window as any).em
em.store.dispatch(async (dispatch: any, getState: any) => {
  const state = getState()
  const a = [em.contextToThoughtId(['Power'])] as Path
  const b = [em.contextToThoughtId(['Kshitigarbha'])] as Path
  if (isContextViewActive(state, a)) {
    dispatch({ type: 'setCursor', path: a })
    dispatch({ type: 'toggleContextView' })
    await sleep(100)
  }
  if (isContextViewActive(state, b)) {
    dispatch({ type: 'setCursor', path: b })
    dispatch({ type: 'toggleContextView' })
    await sleep(100)
  }

  console.log('ACTION: setCursor', prettyPath(state, b))
  dispatch({ type: 'setCursor', path: b })
  console.log('ACTION: toggleContextView (on)', prettyPath(state, b))
  dispatch({ type: 'toggleContextView' })
  await sleep(100)
  console.log('ACTION: toggleContextView (off)', prettyPath(state, b))
  dispatch({ type: 'toggleContextView' })
  await sleep(100)
  console.log('')
  console.log('wait...........')
  console.log('')
  await sleep(1000)
  console.log('ACTION: setCursor', prettyPath(state, a))
  dispatch({ type: 'setCursor', path: a })
  console.log('ACTION: toggleContextView (on)', prettyPath(state, a))
  dispatch({ type: 'toggleContextView' })
})

Current Behavior

Note: Substitute Belonging and 7/3 for Power and Kshitigarbha.

The breadcrumbs for Comunitas only show the Home Context.

image

Expected Behavior

The breadcrumbs should show all the ancestors: 'Articles & Media • The Stoa• Bonnitta Roy • 7/3`

image

@raineorshine raineorshine added the bug Something isn't working label Jan 28, 2025
@raineorshine raineorshine added this to the 📖 Context View I milestone Jan 28, 2025
@raineorshine
Copy link
Contributor Author

#2798 seems to have helped, but I'm still getting missing ancestors on Belonging in my personal thoughtspace.

@raineorshine raineorshine reopened this Jan 29, 2025
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
None yet
Development

Successfully merging a pull request may close this issue.

1 participant