Skip to content

Conversation

@MindFreeze
Copy link
Member

@MindFreeze MindFreeze commented Nov 25, 2025

Proposed change

Fixes this issue #27259 (comment)

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

@MindFreeze MindFreeze requested a review from piitaya November 25, 2025 14:36
@MindFreeze MindFreeze marked this pull request as ready for review November 25, 2025 15:17
@piitaya
Copy link
Member

piitaya commented Nov 25, 2025

CleanShot.2025-11-25.at.16.54.27.mp4

Still the same issue, right?

@piitaya
Copy link
Member

piitaya commented Nov 25, 2025

I think the issue appears because we store /path and not path

bramkragten
bramkragten previously approved these changes Nov 25, 2025
@bramkragten bramkragten dismissed their stale review November 25, 2025 16:03

Missed Pauls message

@piitaya
Copy link
Member

piitaya commented Nov 25, 2025

I can reproduce the issue when there is no path for the view (it is using numeric path)

@piitaya
Copy link
Member

piitaya commented Nov 25, 2025

Replaced by : #28110
@MindFreeze Feel free to reopen your PR if it fix another issue.

@piitaya piitaya closed this Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants