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

Crashes when turning on and off custom currency in account wizard #680

Closed
Krindog7337 opened this issue Sep 1, 2023 · 4 comments
Closed
Labels
bug Something isn't working
Milestone

Comments

@Krindog7337
Copy link
Contributor

Screencast.from.2023-09-01.22-52-27.webm

log :

UnhandledException - unhandled exception: System.NullReferenceException: Object reference not set to an instance of an object.
   at GLib.Internal.StringArrayNullTerminatedSafeHandle..ctor(String[] array)
   at Gtk.StringList.New(String[] strings)
   at NickvisionMoney.GNOME.Views.NewAccountDialog.ValidateCurrency() in /run/build/org.nickvision.money/NickvisionMoney.GNOME/Views/NewAccountDialog.cs:line 385
   at NickvisionMoney.GNOME.Views.NewAccountDialog.<.ctor>b__38_3(Object sender, NotifySignalArgs e) in /run/build/org.nickvision.money/NickvisionMoney.GNOME/Views/NewAccountDialog.cs:line 113
   at GObject.Closure.InternalCallback(IntPtr closure, IntPtr returnValuePtr, UInt32 nParamValues, ValueData[] paramValuesData, IntPtr invocationHint, IntPtr userData)

@nlogozzo nlogozzo added the bug Something isn't working label Sep 1, 2023
@nlogozzo nlogozzo added this to the V2023.9.0 milestone Sep 1, 2023
@nlogozzo
Copy link
Member

nlogozzo commented Sep 2, 2023

I believe this is #644 (comment) which was fixed in main...

Could you install this .flatpak file and run it and confirm if it's fixed or not: https://github.com/NickvisionApps/Denaro/actions/runs/6033236519
Here are the commands to run after you extract the zip:

flatpak install org.nickvision.money.flatpak
flatpak run org.nickvision.money//master

when you are done testing uninstall with flatpak uninstall org.nickvision.money//master

@nlogozzo
Copy link
Member

nlogozzo commented Sep 2, 2023

If that build works, I'll release an update with this as a hot fix

@Krindog7337
Copy link
Contributor Author

I believe this is #644 (comment) which was fixed in main...

Could you install this .flatpak file and run it and confirm if it's fixed or not: https://github.com/NickvisionApps/Denaro/actions/runs/6033236519 Here are the commands to run after you extract the zip:

flatpak install org.nickvision.money.flatpak
flatpak run org.nickvision.money//master

when you are done testing uninstall with flatpak uninstall org.nickvision.money//master

It works now

@nlogozzo nlogozzo closed this as completed Sep 2, 2023
@nlogozzo
Copy link
Member

nlogozzo commented Sep 3, 2023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants