ChickenKyiv / recipe-search-api

https://loopback-recipe-search.herokuapp.com/explorer/#!/Recipe/Recipe_find
GNU Affero General Public License v3.0
2 stars 5 forks source link

remote method for calling data from react form #53

Open atherdon opened 6 years ago

atherdon commented 6 years ago

Based on Nick`s comment https://github.com/ChickenKyiv/recipe-search-react/issues/35#issuecomment-402366514

we need to have one API endpoint in order to populate our search fields

chauhannishith commented 6 years ago

I am unable to understand what is happening in this. Can you explain me what you have done?

atherdon commented 6 years ago

sure, no problem. We should create a new API endpoint, that will return all data, that we need in our search form. so, we'll create a new method at one of the models sample1, sample2, sample3 - sample3 is a crazy method, but i think it'll help you a bit in order to show how i manipulate the data

official docs here: https://loopback.io/doc/en/lb3/Remote-methods.html

chauhannishith commented 6 years ago

I think I didn't explain this properly. I meant to say what is happening currently in this repo, there is no readme so i have no idea. I tried reading the code and scripts but couldnt understand.

Also why do we need to create fetch when we are having a package which omits our hassle of server calls?

atherdon commented 6 years ago

@chauhannishith ohh, got it. I'll prepare an explanation and buzz you today or tomorrow

chauhannishith commented 6 years ago

Can you tell me some basics so I can start understanding this till you tell me in detail

atherdon commented 6 years ago

@chauhannishith i create a simple video for you, please take a look: https://vimeo.com/279601405

chauhannishith commented 6 years ago

can you clear out the table data related to this repository? There are multiple entries for each field

chauhannishith commented 6 years ago

Also now you have made separate files for different ingredients according to their departments i think i will have to make separate functions to combine them and return

or do you have any other plans for it?

chauhannishith commented 6 years ago

npm 19-07-2018 10_35_33

I am trying to get data from package into the DB but i am getting this error. Do you know where this auto update is? Also i deleted the existing data because there were multiple entries for everything

Sorry there are weather issues so the connectivity is still limited

atherdon commented 6 years ago

i get all your messages. just a bit busy. published 4 articles in 1 day :) will reply soon, maybe when wake up, but 0 promises

chauhannishith commented 6 years ago

I still haven't solved this. Can you tell me how you used to run the import file? The database comes as undefined in the import file no matter what i do, I tried keeping the logic in root.js but i am still unable to do this

atherdon commented 6 years ago

Hi Nishith, yeah, i planned to get back to you asap. post this on pasue while i'm not able to help you.

atherdon commented 6 years ago

@chauhannishith ok, i'm opened our server. will review it right now. more comments soon. Btw, jjust to let you know - i've installed groceristar-fetch plugin. so we'll be able to make it work with you together too

atherdon commented 6 years ago

Ok, so looks like our database is clean right now. I start to understand more about your questions

atherdon commented 6 years ago

I started an import script and get an error. will find out and fix it.

Btw, i saw that you've asked a lot of questions - and i didn't answer them. Can you grab that questions into one comment, but put only questions that is still necessary, so i'll be able to anwer them

atherdon commented 6 years ago

@chauhannishith it's not cool to say. but i think we should leave it as is. i tried to debug it, but what i find - that server doesn't return a datasource. you can check my latest commits and see what i've done. i'm a bit angry to loopback, because i think this issue related to their changes. One month ago main server, related to groceristar project started to work strange. and i tried to fix it fast and then gave up and this issue forced me to start a process when we leave our old servers and just migrate to GraphQL approach. what do you think if i'll create a task for you, related to learning how GraphQL works and then we'll rebuild this server together? I think issues from groceristar server and this server are linked, so maybe it's a good time to move to a new, more easier and clean way to do it.

atherdon commented 6 years ago

tell me what do you think

atherdon commented 6 years ago

Hold on. i think i find something

atherdon commented 6 years ago

Nah, it not works... and as i opened main repository - a lot of other issues people posting too. so this is a dead end i assume

chauhannishith commented 6 years ago

Hi @atherdon I think you haven't read my messages on slack. I won't be able to help you at least for some time right now in creating new stuff. But I think MLAB has a feature to export JSON files, you can search it and it might solve your issue for atleast some time right now.

i tried to debug it, but what i find - that server doesn't return a datasource.

i also encountered the same issue and tried to work around in root.js file but didn't work.

atherdon commented 6 years ago

nope, i didn't saw your messages. i think slack should send notification to email, when i'm not at workroom - but i never works fine.

On 30-Jul-2018 at 04:43 AM, Nishith Chauhan wrote:

Hi @atherdon https://github.com/atherdon I think you haven't read my messages on slack. I won't be able to help you at least for some time right now in creating new stuff. But I think MLAB has a feature to export JSON files, you can search it and it might solve your issue for atleast some time right now.

i tried to debug it, but what i find - that server doesn't return a datasource.

i also encountered the same issue and tried to work around in root.js file but didn't work.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ChickenKyiv/recipe-search-api/issues/53#issuecomment-408745343, or mute the thread https://github.com/notifications/unsubscribe-auth/ABZrDpBZZcYbgN1D8oaVO2oJUnGEm3T6ks5uLo8OgaJpZM4VFil7 .