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.

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 a file's title & property drawer's ID #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Wonko7
Copy link

@Wonko7 Wonko7 commented Jun 15, 2024

This seems to do what I want, but I'll be playing with it in the next few weeks, I'll probably find bugs and update this.
In no rush to merge, but I'm open for feedback.

@ndwarshuis
Copy link
Owner

@Wonko7 thanks for working on this. My main feedback so far is that both of the new functions are almost identical to org-sql--insert-alist-add-file-properties so it makes sense to consolidate them.

These additions would also need to be documented, which I can help with unless you also want to take a stab at that.

@Wonko7
Copy link
Author

Wonko7 commented Jun 25, 2024

These additions would also need to be documented, which I can help with unless you also want to take a stab at that.

I'll take a look.

also the code is unfriendly to files missing an ID, I'll push something at some undefined but hopefully soonish point in time.

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.

2 participants