Skip to content

from jotterbot#2

Open
Loogibot wants to merge 13 commits intoLinkedInLearning:mainfrom
Loogibot:loogibot
Open

from jotterbot#2
Loogibot wants to merge 13 commits intoLinkedInLearning:mainfrom
Loogibot:loogibot

Conversation

@Loogibot
Copy link

No description provided.

Loogibot and others added 13 commits February 20, 2025 17:31
…getall implemented , fixed the url for databaseUtils, and accessed the test data in App.java main
…try for the connection and statement objects
…h this project, I don't know enough of the connection, service and postgresql functions to fully understand how its working, notably their respective methods and where/how to use them, unlike Android Room DB I am much more familiar with the DAO system
… mess with the database itself to get this working
…stomer and change the table to wisdom.customers, and prolly a few other things, but I gget it
…added the address field, BUT I did fix the sql commands, notably adding email and phone to the GET_ALL, GET_BY_ID, CREATE, UPDATE, and DELETE strings in CustomerDao. You need to add all of the relevant column names for each sql command, of which I'm still getting the hang of, it seems to be specifically a postgresql formatting thing, especially since the mehods need to have a matching parameter, tho some of them don't need everying like in getOne but the GET_ALL string has all of the fields from the Customer class, note still without the address column
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.

1 participant