-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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.
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
Add: org-journal support #7060
Add: org-journal support #7060
Conversation
|
||
(defun org/init-org-journal () | ||
(use-package org-journal | ||
:if org-enable-org-journal-support |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks redundant when you are using :toggle
for package selection.
Anything else that I should fix for this? |
Maintainers will review it further before it gets added to |
Ok, no problem thanks |
bump. will this be ok to be merged? seemingly a great feature. |
@nickanderson thanks for your work on this. Would you mind rebasing on the current develop branch and resolving the merge conflicts? |
62486c4
to
41096e7
Compare
Surething |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested this locally and it worked like a charm. I also updated my spacemacs-tmux docker image to use this branch, so if you're interested in testing it without altering your local spacemacs config try the following:
docker run -it quay.io/jgmize/spacemacs-tmux
|
||
(setq org-journal-dir "~/org/journal/") | ||
(setq org-journal-file-format "%Y-%m-%d")) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, so this org-journal-dir
and org-journal-file-format
are my own preferences, they seem to have snuk in. Shall I remove those so that they can be set in user-config?
The default value for org-journal-dir is ~/Documents/journal/
and the default file format is YYYYMMDD
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like your defaults better than the package defaults; perhaps just add docs for how to configure them in .spacemacs
?
fd34752
to
0465e8c
Compare
Rebased, removed my defaults updated docs. |
I tested out your latest updates, including setting the org-journal-dir and org-journal-file-format variables and I got the following error when I opened a new journal entry with
|
Well that's a shame. @jgmize Did you verify that org-journal-dir existed? |
Yes, and it opened an existing journal file I had created earlier. I should have mentioned that the message was a warning than a blocker, after dismissing it things worked as expected. |
Ok, so some keybinding didnt get automatically remapped. I don't see it as an issue, but warnings are annoying. Is there some way to avoid it? |
0465e8c
to
bb635de
Compare
I aligned the new journal entry keybindings. Now its |
The latest changes have been working well for me: no more warnings and the new keybindings work as expected. Thanks again for your work on this @nickanderson, and hopefully someone with commit access will be able to take a look at this before another merge conflict is introduced. |
Great, thanks for the feedback @jgmize. I can always re-base if a conflict arises :). |
And also fix indentation. This is to complete #7060
Thank you 👍 Cherry-picked into Folks, go ahead and enjoy 😸 I've made several changes in 0477040:
|
thanks @d12frosted ! |
And also fix indentation. This is to complete syl20bnr#7060
And also fix indentation. This is to complete syl20bnr#7060
No description provided.