mirror of
https://github.com/BoostIO/BoostNote-App.git
synced 2026-04-27 05:15:56 +03:00
[PR #1560] [MERGED] remove page{Doc|Folder|Workspace} if exists in pageProps merge data #1445
Labels
No labels
android 🤖
assigned to core 🦹
bug 🐛
documentation 📚
documentation 📚
duplicate 🚫
external issue 🔼
external issue 🔼
feature request 🌟
funded on issuehunt 💵
help wanted 🆘
improvement request 🔨
improvement request 🔨
ios 🍎
mobile 📱
needs investigation 🔬
needs more info ℹ️
needs specs 📐
plugin idea 🔌
plugin idea 🔌
poll 🗳️
pull-request
question ❓
rewarded on issuehunt 🎁
security issue 🔑
won’t fix ❌
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/BoostNote-App#1445
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/BoostIO/BoostNote-App/pull/1560
Author: @ButteryCrumpet
Created: 5/16/2022
Status: ✅ Merged
Merged: 5/16/2022
Merged by: @ButteryCrumpet
Base:
master← Head:bug/only-default-workspace-breadcrumb📝 Commits (1)
53a0442remove page{Doc|Folder|Workspace} if exists in pageProps merge data📊 Changes
1 file changed (+21 additions, -3 deletions)
View changed files
📝
src/cloud/lib/stores/pageStore/store.tsx(+21 -3)📄 Description
Issue
Breadcrumbs would show only last visited workspace on every page.
Cause
Merging new
pagePropsinto existing would not remove oldpage{Doc|Folder|Workspace}if it was a page transition.Fix
Remove old
page{Doc|Folder|Workspace}ifpage{Doc|Folder|Workspace}exists in new merge data🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.