org-ql icon indicating copy to clipboard operation
org-ql copied to clipboard

Fix: (org-ql-view.el) Adjust priority for agenda elements

Open anpandey opened this issue 2 years ago • 19 comments

Fixes #38.

anpandey avatar Jan 22 '23 07:01 anpandey

Thanks for submitting this. I'll plan to merge it for v0.8.

alphapapa avatar Mar 14 '23 09:03 alphapapa

@anpandey: I made a pull request into your repository that fixes the merge conflict. Merging it there should make the patch ready to merge here too.

larrasket avatar Jan 25 '24 18:01 larrasket

Update: seems like even after syncing the branch with the upstream here, this patch no longer works (the entities are not sorted correctly).

larrasket @.***> writes:

@anpandey: I made a pull request into your repository that fixes the merge conflict. Merging it there should make the patch ready to merge here too.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.

-- Regards, Salih

larrasket avatar Jan 25 '24 20:01 larrasket

Update: seems like even after syncing the branch with the upstream here, this patch no longer works (the entities are not sorted correctly).

Be sure you test in a clean Emacs configuration, e.g. using https://github.com/alphapapa/makem.sh (load up makem.el and call M-x makem, then use a sandbox configuration and the interactive rule to run the checked-out branch in a clean, sandboxed configuration.

alphapapa avatar Jan 25 '24 21:01 alphapapa

It didn't work either when I tried that time. I tried now, after upgrading org version [now: Org mode version 9.6.17 (9.6.17-??-25c786f] it worked both in the sandbox and with my main configuration.

larrasket avatar Feb 05 '24 20:02 larrasket

It didn't work either when I tried that time. I tried now, after upgrading org version [now: Org mode version 9.6.17 (9.6.17-??-25c786f] it worked both in the sandbox and with my main configuration.

Ok, so it does work now, with the latest, rebased patch and the latest Org version?

alphapapa avatar Feb 06 '24 06:02 alphapapa

Adam Porter @.***> writes:

Ok, so it does work now, with the latest, rebased patch and the latest Org version?

Yes, I can confirm.

-- Regards, Salih

larrasket avatar Feb 07 '24 00:02 larrasket

@anpandey By the way, we're hoping to merge org-ql into Org mode, which means that FSF copyright assignment comes into play. AFAICT this patch plus your other one that was committed (i.e. not including comments) would be 12 lines, which is allowable without doing assignment. But it would be good to do the CA anyway. Have you done this already for Emacs/Org, or would you be willing to?

alphapapa avatar Feb 22 '24 01:02 alphapapa

@alphapapa That's great news! I have not signed any CA before, but I can confirm my employer will allow it (as long as the assignment is to the FSF).

anpandey avatar Feb 22 '24 07:02 anpandey

@alphapapa That's great news! I have not signed any CA before, but I can confirm my employer will allow it (as long as the assignment is to the FSF).

Great. You can send an email to [email protected] or to [email protected] to request the assignment form. Please let me know when you've submitted it and when you receive the confirmation (it usually takes a few weeks, depending on circumstances).

alphapapa avatar Feb 22 '24 07:02 alphapapa

@alphapapa I've received the confirmation of the copyright assignment to the FSF.

anpandey avatar Mar 14 '24 22:03 anpandey

@alphapapa I've received the confirmation of the copyright assignment to the FSF.

Thanks. I recently found out from Eli Zaretskii that I'm supposed to get confirmation of this directly from either the FSF clerk or an Emacs maintainer before accepting contributions. Would you be willing to ask the clerk to email confirmation to me directly?

alphapapa avatar Mar 15 '24 00:03 alphapapa

No problem, I've asked the copyright clerk to email you the confirmation at the address on your profile.

anpandey avatar Mar 15 '24 02:03 anpandey

Thanks.

alphapapa avatar Mar 15 '24 03:03 alphapapa

@alphapapa You should have gotten a confirmation email from the FSF copyright clerk.

anpandey avatar Apr 11 '24 04:04 anpandey

@alphapapa You should have gotten a confirmation email from the FSF copyright clerk.

@anpandey I'm not sure if I did. Anyway, Eli told me further that what I'm supposed to do is ask him or one of the other Emacs maintainers to verify that your name and email address are on their list. Would you please let me know exactly what name and address to ask him about? Thanks.

alphapapa avatar Aug 22 '24 09:08 alphapapa

Sure, the name should be 'Ankit Raj Pandey', and the email '[email protected]'.

anpandey avatar Aug 28 '24 00:08 anpandey