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

"Note To Self" #560

Open
EdGeraghty opened this issue Feb 8, 2022 · 4 comments · May be fixed by #569
Open

"Note To Self" #560

EdGeraghty opened this issue Feb 8, 2022 · 4 comments · May be fixed by #569
Assignees
@EdGeraghty
Labels
approved Current description and details of feature have been approved for dev by core team feature New feature or request

Comments

@EdGeraghty
Copy link
Collaborator

Much like in Signal, I like to set up & pin a "Note to Self" room on Matrix. Is this worth offering as an opt-in feature?

I guess people can create the room themselves, but TBH it wouldn't occur to me to do so if not offered.

Thoughts?
@ereio
Copy link
Member

ereio commented Feb 8, 2022

Would love to add it; I do the same in Signal. Would follow the same UX as Signal for now.

I approve that flow 👍

@ereio ereio added approved Current description and details of feature have been approved for dev by core team feature New feature or request labels Feb 8, 2022
@ereio ereio changed the title Feature: "Note to Self" "Note To Self" Feb 8, 2022
@EdGeraghty EdGeraghty self-assigned this Feb 9, 2022
@EdGeraghty
Copy link
Collaborator Author

image

Just to double-check, how do we want to do this? It looks like Syphon already lists the account as first in its search list - are we literally changing that to "Note to Self" instead of displaying... the profile name?

@ereio
Copy link
Member

ereio commented Feb 10, 2022

Yes, if that user == the current user, display "Note To Self"

Open to suggestions here though if that would cause any regrettable complexity

@EdGeraghty
Copy link
Collaborator Author

EdGeraghty commented Feb 15, 2022 •
edited
Loading

#569

Turns out it's not quite as simple as I'd hoped! I'll move comments & etc to the PR.

@EdGeraghty EdGeraghty linked a pull request Apr 20, 2022 that will close this issue
Open
17 tasks
@EdGeraghty EdGeraghty linked a pull request Apr 22, 2022 that will close this issue
[feature] Note To Self #569
Open
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@EdGeraghty EdGeraghty

Labels
approved Current description and details of feature have been approved for dev by core team feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[feature] Note To Self EdGeraghty/syphon
2 participants
@ereio @EdGeraghty