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

#### [Groceristar][Favorites][methods] #3

Closed
atherdon opened this issue Feb 1, 2019 · 13 comments
Closed

#### [Groceristar][Favorites][methods] #3

atherdon opened this issue Feb 1, 2019 · 13 comments

Comments

@atherdon
Copy link
Member

atherdon commented Feb 1, 2019

By using names on methods from this page
In order to make it better, we'll create a set of constants, each for a different method.

Example:
Get All ingredients with status favorite
will became export const GET_ALL_FAVORITE_INGREDIENTS = "GET_ALL_FAVORITE_INGREDIENTS ";

@kursadcayir
Copy link

for Delete ingredients from favorites list , export const DELETE__FAVORITE_INGREDIENTS_IN_LIST = "DELETE__FAVORITE_INGREDIENTS_IN_LIST";

@kursadcayir
Copy link

for addition of color ,
export const INGREDIENT_ADD_FAVORITE_COLOR = "INGREDIENT_ADD_FAVORITE_COLOR ";

for clearing the color
export const INGREDIENT_CLEARING_COLOR = "ADD_FAVORITE_COLOR ";

@kursadcayir
Copy link

for method that list all favorites;
export cons LIST_FAVORıTES= "LIST_FAVORıTES ";

@atherdon
Copy link
Member Author

atherdon commented Feb 15, 2019 via email

@kursadcayir
Copy link

Actually , I did not understand what to do . Do I need to implement these variables in table ?

@atherdon
Copy link
Member Author

atherdon commented Feb 15, 2019 via email

@kursadcayir
Copy link

Sorry. I think that I am gonig to implement documentation. my bad .

@atherdon
Copy link
Member Author

not a problem at all. do you know how to contribute to github repositories? i have a good tutorial for that - just tell me if you need it

@atherdon
Copy link
Member Author

@kursadcayir how its going with this task? can you push changes that you have so far?

@kursadcayir
Copy link

Hi ,
I did not focus on much about graph ql , because of the job search . I will try to push in this week .

@atherdon
Copy link
Member Author

you can tell me if you busy - it's ok, and i'll move this task to someone else

@atherdon
Copy link
Member Author

just have an idea - maybe i need to share your linkedin profile at my network - maybe it will help you?

@atherdon
Copy link
Member Author

tell me what do you think

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants