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.

Sign up for GitHub

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

Jump to bottom

Fix for #1639 journal overwrite #1640

Closed
wants to merge 5 commits into from
Closed

Fix for #1639 journal overwrite #1640

wants to merge 5 commits into from

Commits on Nov 11, 2022

  1. Bail out of search_mode before editing (rather than after) if no entr… 

    …ies match the search
    @richardjs
    richardjs committed Nov 11, 2022
    Configuration menu
    Copy the full SHA
    fce5aff View commit details
    Browse the repository at this point in the history

  2. Add test for jrnl-org#1639

    @richardjs
    richardjs committed Nov 11, 2022
    Configuration menu
    Copy the full SHA
    37377b9 View commit details
    Browse the repository at this point in the history

  3. Refine jrnl-org#1639 fix to account for established use cases (and te… 

    …sts) of calling --edit on an empty journal
    @richardjs
    richardjs committed Nov 11, 2022
    Configuration menu
    Copy the full SHA
    35b93f1 View commit details
    Browse the repository at this point in the history

Commits on Nov 13, 2022

  1. Adjust fix for jrnl-org#1639 to still create entries if both --edit a… 

    …nd --change-time flags are present
    @richardjs
    richardjs committed Nov 13, 2022
    Configuration menu
    Copy the full SHA
    0fbdf25 View commit details
    Browse the repository at this point in the history

  2. Change time on the new entry when --change-time and --edit are used t… 

    …ogether with a search that matching no entries
    @richardjs
    richardjs committed Nov 13, 2022
    Configuration menu
    Copy the full SHA
    3bd76fa View commit details
    Browse the repository at this point in the history