-
Notifications
You must be signed in to change notification settings - Fork 46
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
Migrate into updated Vite structure #578
Comments
Fixed multiple Ynab issues. Ynab works now |
Fixed discord link |
Now need to inject the Google client ID and client secret into the pipeline. BTW, I failed to download the draft artifacts, don't know why, but in case you need another way, use this:
Done |
@baruchiro lint is working now so you can cross it out :) |
Part of #578 --------- Co-authored-by: Jonathan <brafdlog@gmail.com> Co-authored-by: Shai Ungar <shaiungar@gmail.com>
@baruchiro what solved this issue? I still see this issue when installing from the latest released version
|
@brafdlog I also see it. I'm debugging it now |
@brafdlog I found the issue. |
Nice, thanks @shaiu |
@brafdlog @baruchiro fixed tests + lint + typecheck: #595 |
Issue has been marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
We have many deprecated dependencies and can't upgrade the
israeli-bank-scrapers
anymore because it upgraded its NodeJS version.I migrated our code into a new updated template, with the latest technologies and a better project structure to support us in the future.
PR: #579
Here is what is missing to finish the migration:
Critical
electron-builder
as we do, or something else?)Can be fixed after migration
Close #428
The text was updated successfully, but these errors were encountered: