Skip to content

Feat/matching prompts#31

Open
WoosterInitiative wants to merge 24 commits intoDanielKarp:mainfrom
WoosterTech:feat/matching_prompts
Open

Feat/matching prompts#31
WoosterInitiative wants to merge 24 commits intoDanielKarp:mainfrom
WoosterTech:feat/matching_prompts

Conversation

@WoosterInitiative
Copy link
Collaborator

This is still a work in progress but I want to make sure everyone else likes the direction I'm heading before finalizing all of it.

  • I've added tox to test against all supported Python versions. This helps make sure we aren't using features that aren't yet present in older versions.
  • I've added a contribution guide to make sure that a few of the standards are followed, specifically that pre-commit is used to enforce formatting standards on any new changes.

The main goal is actually the new new-ynamazon cli tool. The name is just a stand-in, but it does a few things:

  1. A config.toml file can be passed in. This is helpful for multiple Amazon accounts (husband/wife/etc.).
  2. Allows the user to select from a list of matching transactions. Doesn't do any fancy matching, fuzzy or otherwise, but does sort by order date.

Please take a look. I'm not an expert and pull requests, but you could perhaps clone the branch from my repo? I'm sure there's a better way, I just don't know it.

Please leave any feedback and I'll be glad to make any changes.

Karl Wooster and others added 23 commits April 17, 2025 16:57
…o `_fetch_amazon_order_history`

fix tests to properly support this
…project passes all tox tests

required eliminating some "newer" features such as `X
Y` unions and match-case blocks
need to add tests for cli since that is where most people will use the code
@WoosterInitiative
Copy link
Collaborator Author

Creates a solution for #10 , though it is pretty naive.

… and locking of dependencies

remove uv-export hook from pre-commit and autoupdate all repos
@WoosterInitiative
Copy link
Collaborator Author

Another one: #19

@WoosterInitiative WoosterInitiative marked this pull request as ready for review April 29, 2025 14:38
@omarshahine
Copy link
Contributor

I'm excited about multiple account support.

@WoosterInitiative
Copy link
Collaborator Author

@DanielKarp hoping to get some feedback on this idea. I'm not ready to merge it but I'm waiting for feedback before finishing it up

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