hql287 / Manta

🎉 Flexible invoicing desktop app with beautiful & customizable templates.
https://www.getmanta.app
GNU Lesser General Public License v3.0
5.28k stars 478 forks source link

Can't change language without setting tax amount greater than zero first #273

Closed romainlq closed 6 years ago

romainlq commented 6 years ago

Expected Behavior

When I launch the app, I should be able to switch language without doing anything else

Current Behavior

When starting the app for the first time, Tax amount is set to zero. So, I get an "Invalid tax" error while just trying to switch language.

Possible Solution

Steps to Reproduce (for bugs)

  1. Launch app for the first time
  2. Go to Settings -> General
  3. Switch language to whatever language
  4. Click on the Save button

Context

Just discovered this app and would love to contribute ! I just started the app in dev mode for the first time, wanted to check in the options and switch language, but then when I wanted to save, I got the "invalid tax" warning.

Your Environment

romainlq commented 6 years ago

How to reproduce if the app already installed :

hql287 commented 6 years ago

@romainwn: Thanks for reporting this.

Actually, you can't save the settings when the tax amount is equal or less than zero. This is the default value for the tax field (and will only matter) if you decide to use it in the form. So just set it to something greater than 0 and you're good to go.