Skip to content

feat: conversation link button in memory items - #2988

Merged
beastoin merged 3 commits into
BasedHardware:mainfrom
krushnarout:feat/memories
Sep 17, 2025
Merged

feat: conversation link button in memory items#2988
beastoin merged 3 commits into
BasedHardware:mainfrom
krushnarout:feat/memories

Conversation

@krushnarout

Copy link
Copy Markdown
Member

closes #2889

Screen.Recording.2025-09-13.at.4.21.31.PM.mov

@beastoin

Copy link
Copy Markdown
Collaborator
Screenshot 2025-09-16 at 11 15 37

what is this? review your code before opening the PR, or just mark it as draft if it’s not ready.

@krushnarout

@krushnarout

Copy link
Copy Markdown
Member Author

what is this? review your code before opening the PR, or just mark it as draft if it’s not ready.

@beastoin I mistakenly included changes of a different approach, thank you

@beastoin
beastoin merged commit d6a8cf3 into BasedHardware:main Sep 17, 2025
@beastoin

Copy link
Copy Markdown
Collaborator

lgtm @krushnarout

@beastoin

Copy link
Copy Markdown
Collaborator

reverted, pls resolve #3000

my question, did you see the build error ?

@krushnarout

@krushnarout

Copy link
Copy Markdown
Member Author

reverted, pls resolve #3000

my question, did you see the build error ?

No, before every pull request, I use flutter analyze. How do I check for build errors ?

@beastoin

@beastoin

Copy link
Copy Markdown
Collaborator

sry, it’s a runtime exception. did you try running it and showing the exception during rendering?

check your console logs.

@krushnarout

@krushnarout

krushnarout commented Sep 18, 2025

Copy link
Copy Markdown
Member Author

before it was working fine #2988 (comment)

Screen.Recording.2025-09-13.at.4.21.31.PM.mov

after I added the changes from #2988 to the latest main branch, I see so many errors, and also it's broken.

Screen.Recording.2025-09-18.at.10.57.36.AM.mov

fixed in pr #3005

@beastoin

@krushnarout
krushnarout deleted the feat/memories branch September 18, 2025 06:13
Glucksberg pushed a commit to Glucksberg/omi-local that referenced this pull request Apr 28, 2026
* feat: conversation link button to memory items

* remove unrelated changes
Glucksberg pushed a commit to Glucksberg/omi-local that referenced this pull request Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

memories should be linked to conversations

2 participants