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

fix amanuensis create to create separate contracts #189

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

Conversation

dbosk
Copy link
Owner

@dbosk dbosk commented Oct 29, 2024

One TA had a contract until 2024-10-27. I later wanted to create a new contract
from 2024-10-28. But that new contract included the old contract's events. It
shouldn't, it should only contain events from 2024-10-28 onwards.

Other changes:

  • Clarifies docstring for the amanuensis create command
  • Removes unneeded .copy() from filter_events_by_date function

dbosk added 2 commits October 29, 2024 10:55

Verified

This commit was signed with the committer’s verified signature.
dbosk Daniel Bosk

Verified

This commit was signed with the committer’s verified signature.
dbosk Daniel Bosk
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.

None yet

1 participant