Added Nextcloud Notes support for notes and shopping lists#371
Added Nextcloud Notes support for notes and shopping lists#371tylxr59 wants to merge 3 commits intoStypox:masterfrom
Conversation
app/src/main/kotlin/org/stypox/dicio/skills/nextcloud_notes/NextcloudNotesSkill.kt
Outdated
Show resolved
Hide resolved
app/src/main/kotlin/org/stypox/dicio/skills/nextcloud_notes/NextcloudNotesSkill.kt
Outdated
Show resolved
Hide resolved
| }, | ||
| ) | ||
|
|
||
| StringSetting( |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Could do something like dicio-{timestamp}.md as a default but adding a .note_topic. to the sentences for specific titles
There was a problem hiding this comment.
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
app/src/main/kotlin/org/stypox/dicio/skills/nextcloud_notes/NextcloudNotesOutput.kt
Outdated
Show resolved
Hide resolved
app/src/main/kotlin/org/stypox/dicio/skills/nextcloud_notes/NextcloudNotesSkill.kt
Outdated
Show resolved
Hide resolved
app/src/main/kotlin/org/stypox/dicio/skills/nextcloud_notes/NextcloudNotesSkill.kt
Outdated
Show resolved
Hide resolved
2 participants
Added support for taking notes and adding items to a shopping list and saving them to Nextcloud.
Resolves #199