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

chore: SQLiteOneDriveInvoice Sample - Update to net6 #219

Merged
merged 8 commits into from
Jul 19, 2023

Conversation

aesteves900
Copy link
Contributor

No description provided.

@nickrandolph
Copy link
Collaborator

@aesteves900 there's a bunch of small bug fixes that need to be done but you'll need to make sure each project has the correct nuget references - this is a SQLite sample, so you should expect that all the projects reference sqlite.

@aesteves900 aesteves900 force-pushed the dev/AEST/190-update-to-net6-sqllite-inv branch from 6ba4a41 to a55cffc Compare November 20, 2022 23:01
@aesteves900 aesteves900 marked this pull request as ready for review November 20, 2022 23:18
@nickrandolph nickrandolph marked this pull request as draft November 24, 2022 07:02
@nickrandolph
Copy link
Collaborator

@aesteves900 I'm pushing this back to draft:

  • Sample doesn't run on Windows (I haven't tested other targets)
  • Please update WindowsApp Sdk to latest
  • Please add Gtk/Wpf/Linux targets unless they're not supported

@aesteves900 aesteves900 force-pushed the dev/AEST/190-update-to-net6-sqllite-inv branch 2 times, most recently from cc27475 to 90b05ab Compare November 28, 2022 20:38
@aesteves900

This comment was marked as resolved.

@aesteves900 aesteves900 self-assigned this Nov 30, 2022
@aesteves900 aesteves900 marked this pull request as ready for review December 2, 2022 16:13
@aesteves900 aesteves900 marked this pull request as draft December 6, 2022 02:21
@aesteves900
Copy link
Contributor Author

Hello @nickrandolph and @jeromelaban , the issue about the SqlLite on WASM is:

The type initializer for 'SQLite.SQLiteConnection' threw an exception.
---> System.DllNotFoundException: e_sqlite3
at SQLitePCL.SQLite3Provider_e_sqlite3.SQLitePCL.ISQLite3Provider.sqlite3_libversion_number()
at SQLitePCL.raw.SetProvider(ISQLite3Provider imp)
at SQLitePCL.Batteries_V2.Init()
at SQLite.SQLiteConnection..cctor() in /Users/vagrant/git/src/SQLite.cs:line 169

The path from Demo.db file is obtained by

private readonly string databasePath = Path.Combine(Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.LocalApplicationData), "Demo.db"));

That's fails exactlly here:

using (var connection = new SQLiteConnection(databasePath)) {

@jeromelaban
Copy link
Member

Thanks. It's likely that this demo never worked on wasm, it's missing the Uno.sqlite-wasm package. Can you validate if it did without your changes?

@aesteves900
Copy link
Contributor Author

Thanks. It's likely that this demo never worked on wasm, it's missing the Uno.sqlite-wasm package. Can you validate if it did without your changes?

Not working as well, there is the same error.

@aesteves900
Copy link
Contributor Author

@jeromelaban , I added up Uno.sqlite-wasm and Uno.SQLitePCLRaw.provider.wasm, but still in issue.

'SQLite.SQLiteConnection' threw an exception. ---> System.TypeLoadException: Could not load list of method overrides due to Method not found: int SQLitePCL.ISQLite3Provider.sqlite3_win32_set_directory(int,string)

@jeromelaban
Copy link
Member

Thanks. Let's keep the sample as-is then. As long as the other targets work for now it's fine. Let's open an issue to fix it, still.

@aesteves900
Copy link
Contributor Author

The issue was created to solve the SQLLite problem on WASM platform. #258

@aesteves900 aesteves900 marked this pull request as ready for review December 6, 2022 20:27
@nickrandolph
Copy link
Collaborator

@aesteves900 more work required on this sample

  • Windows doesn't run - I've pushed a fix that will allow it to run but still needs work to tidy up
  • Does the OneDrive integration work?
  • Why are these files not included in the Shared project

image

@nickrandolph nickrandolph marked this pull request as draft December 9, 2022 03:25
@nickrandolph nickrandolph reopened this Jul 17, 2023
@nickrandolph nickrandolph mentioned this pull request Jul 17, 2023
16 tasks
aesteves900 and others added 4 commits July 19, 2023 18:43
chore: SQLiteOneDriveInvoice Sample - Update to net6

chore: SQLiteOneDriveInvoice Sample - Update to net6

chore: SQLiteOneDriveInvoice Sample - Update to net6

fix: Making small corrections
@nickrandolph nickrandolph marked this pull request as ready for review July 19, 2023 08:51
@nickrandolph nickrandolph force-pushed the dev/AEST/190-update-to-net6-sqllite-inv branch from d3c4cd5 to 3152705 Compare July 19, 2023 08:51
@nickrandolph nickrandolph merged commit 69c0afd into master Jul 19, 2023
6 checks passed
@nickrandolph nickrandolph deleted the dev/AEST/190-update-to-net6-sqllite-inv branch July 19, 2023 09:11
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.

None yet

3 participants