enm10k / nocodb-mobile

A prototype Android/iOS client for NocoDB
MIT License
46 stars 5 forks source link

Compatibility with NocoDB >= v0.200 #2

Open patschwork opened 1 year ago

patschwork commented 1 year ago

A login with the app is now possible (thank you for resolving #1 :-) ).

With the new NocoDB release it seem major changes in the API -> https://github.com/nocodb/nocodb/releases/tag/0.202.4

enm10k commented 1 year ago

Thank you for letting me know that your issue has been resolved and for the update of NocoDB. I will check for the API compatibility issues.

Additionally, it appears that the new update includes responsive support. I'd like to check out that feature.

patschwork commented 1 year ago

Additionally, it appears that the new update includes responsive support.

Yes, it has been improved. But (currently) the viewing of datasets is ok-ish. Editing is not possible (tested on Android with Firefox).

imho: A native app (Android, iOS, ...) is always better for the UI/UX

enm10k commented 1 year ago

@patschwork I have started making fixes here and have already fixed the bug you reported. I am currently checking to see if there are any other issues.

I would appreciate it if you could try the latest binary and give me your feedback.

patschwork commented 1 year ago

@patschwork I have started making fixes here and have already fixed the bug you reported. I am currently checking to see if there are any other issues.

I would appreciate it if you could try the latest binary and give me your feedback.

Hi @enm10k, quick testing: In general it's working 🥳

Thank you! 🙂

enm10k commented 1 year ago

Hi @patschwork, Thank you for checking the fix.

  • Some tables don't show the content (no error message). Will try this later again and give an update here...

The issue with content not displaying is likely due to the schema I am using to parse HTTP responses being incorrect. I'll also check to see if the same issue occurs with my test database.

Additionally, I'm going to implement a mechanism to display such crashes as error messages.

enm10k commented 12 months ago

Merged #3 that partially solves the issue.