Skip to content

Conversation

@tylxr59
Copy link

@tylxr59 tylxr59 commented Dec 12, 2025

Added support for taking notes and adding items to a shopping list and saving them to Nextcloud.

Resolves #199

},
)

StringSetting(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure that having a setting for note names is the best way of doing it. I think it might be better to pass the note name as a input value and have a setting for either choosing the file extension or a default note name. That way, if someone wants to make a new note or a note about a different topic, they won't have to stick it in one main note file.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could do something like dicio-{timestamp}.md as a default but adding a .note_topic. to the sentences for specific titles

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't fully resolve this in my latest commit. I tried messing around with various implementations of this and I wasn't happy with what I ended up with. I think trying to make sentences with .topic. in it makes it hard to cover a lot of natural speech.

There may be a better way to do this and I'll revisit it some time soon

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it might be a good idea to add an option to retrieve the grocery list. Right now we can only add, but it could be good to have it read it to you.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ill do some research on it this morning

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Implemented in latest commit. Also added a way to check if specific items are already on the list.

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.

Text notes skill

2 participants