Receipt-Wrangler / receipt-wrangler-api

Server for Receipt Wrangler
GNU Affero General Public License v3.0
83 stars 1 forks source link

Feature Suggestions #211

Closed queball1999 closed 7 months ago

queball1999 commented 7 months ago

I may try my hand at implementing some of these features but I also wanted to share them as suggestions for others to explore.

So far I have really liked how simple and intuitive the software is and would very much like to contribute to this project. However, as of now my strongest programming language is Python so I will try my best. Feel free to leave feedback.

Noah231515 commented 7 months ago

Hey thanks a bunch for the feedback! If you need help with contributions, let me know! I'd love to help.

  1. Navigating to the list after saving makes a lot of sense. This for sure can be the pattern across the app. I will make an issue on the desktop to implement this.
  2. In which cases is this most frustrating? Perhaps this could be configurable, since this is useful when adding multiple categories/tags on a receipt. Its tricky because its useful when you want to add more than one, but not as useful when you only want to add one.
  3. Agreed. I'll do some initial research into this.
  4. Take a look at https://receiptwrangler.io/docs/concepts/groups/overview#what-is-the-all-group, maybe this will help clear things up. Adding something in app could be useful here.
  5. I will have to think about this a bit more. In the long run it probably is better to have these system settings configured via an interface. It is friendlier to do so and seems like a natural progression from config file driven settings to db driven settings. I did it this way initially to keep it simple, but it is time to start thinking more about ease of use.
  6. Agreed, dashboards have yet to be really fleshed out yet.
  7. Agreed, this is in the backlog.
  8. What do you mean the currency is integers? (This may have been due to a bug with the input mask that was fixed today)
  9. I can help you out with that. If you haven't done so already, read over the docs at https://receiptwrangler.io/docs/configuration/ on the AISettings section. What does your config look like atm?
queball1999 commented 7 months ago

I will be reviewing everything and trying to make some small modifications before anything serious. I will definitely reach out if I have any questions.

Regarding your response:

  1. Awesome!
  2. Hmm, maybe if the mouse is detected in the drop-down menu it stays open, and if it goes outside, the menu will close? I think this may accommodate both use both cases because if I want one, I just have to select it and move my mouse out of the way.
  3. Awesome!
  4. Will do.
  5. No worries, I understand your concerns. Feel free to stick with what you feel is best.
  6. I suspected as much, nevertheless, it is always good to document features to be added.
  7. Awesome!
  8. For some odd reason, I can only define amounts as integers or whole numbers instead of floating-point integers. This causes the split totals of odd numbers to carry over the decimals, as in the image below. I entered the amount of $21 so it should be $10.5 for each and not $105. image
  9. My AI config is as follows and it doesn't seem to work. I have blanked the openAI key but have verified it is valid. I want to start initially using openAI and then eventually switch to self-hosted. image
Noah231515 commented 7 months ago

2. Yeah I'll see what other apps do too. The way it is implemented now has always felt weird to me, but worked.

8. This issue was fixed in the latest PR in desktop. Get the latest version to have this fixed.

9. Do you post the contents of logs/app.log ?

excuse the formatting lol

queball1999 commented 7 months ago
  1. I was able to get this resolved by pulling the latest release. Thank you!
  2. I still have not gotten this working and was unable to locate any logs. If I do docker logs I do not see anything, and if I navigate to /etc/log I still do not see anything.
Noah231515 commented 7 months ago

Two things to check:

  1. Check your docker-compose.yaml to see if the logs volume is mounted. That is where they will be. For example, in https://receiptwrangler.io/docs/configuration-examples/mariadb relative to the directory that docker-compose.yaml is in, the logs will be found at ./logs/app.log
  2. Have you purchased OpenAI usage tokens? They recently changed their API usage to require pre purchased tokens instead of pay as you go.
queball1999 commented 7 months ago

Here is an image of my config; I am running the monolithic container. Initially the ./logs:/app/receipt-wrnagler-api/logs was missing and after I added it, I was able to access the logs.

image

Seems like my openAI key is wrong! After adding a new payment method and refreshing my key, I got everything working.

Noah231515 commented 7 months ago

Awesome, glad you got it working! I'll close this issue out when I've finished up making backlog items for some of suggestions.

Thanks again for the feedback!

queball1999 commented 7 months ago

No worries, I appreciate your help.

You can continue to close this out but I did notice another item I thought I'd share:

Again, I will hopefully start to contribute to this project. I am still getting the feel for everything and reviewing what you have already done. Thanks again for al the help!!

Noah231515 commented 7 months ago

Love this idea, adding a backlog item for it.