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

Explain/Create Initial Database State for New Projects #23

Open
kentwills opened this issue Feb 21, 2017 · 9 comments
Open

Explain/Create Initial Database State for New Projects #23

kentwills opened this issue Feb 21, 2017 · 9 comments

Comments

@kentwills
Copy link
Contributor

For those unfamiliar with Google App Engine, there is a bit of a hurdle to go from the current project to something that can be used quickly. All database manipulations happen through the App Engine Datastore portal. We should either give a quick tutorial for initial state or provide a script to populate initial state.

@elg0nz
Copy link

elg0nz commented Feb 27, 2017

Any updates on adding these instruction?
I've managed to build and push the app to Google App Engine. But even after updating the employees json and running the populate_employees task, I only see the Darwin Yelp user.

Also whenever I try to make a new meeting request, I get back a 500 error.
I'll be happy to help writing a bootstrapping script if you can provide a "SQL dump" or equivalent of the desired initial state.

@kentwills
Copy link
Contributor Author

@elg0nz It's great to hear that you got the application up and running. I set up a slack channel @ yelp-beans.slack.com where we can help you work through individual problems with stack traces. Maybe we can work through your problem specifically and see what patches / additions to documentation we should add?

@elg0nz
Copy link

elg0nz commented Feb 27, 2017

Sounds great, could you send me an invite to the email on my profile? (@elg0nz) Cheers!

@kentwills
Copy link
Contributor Author

sent!

@bxm156
Copy link
Member

bxm156 commented Feb 28, 2017

@kentwills Should we make that slack channel public?

@elg0nz You will only see your user on the homepage. Did you authorize the application with the same google email that matches your employee record that is in the employees.json file?

@elg0nz
Copy link

elg0nz commented Feb 28, 2017

@bxm156 Kent and were looking into this issue this morning, and this is what we came up with:

In order for a user to be displayed & able to request a meeting the following must happen:

  1. The employee row from the JSON dict must include a location param.
    i.e. {"first_name": "Bob", "last_name": "Foobar", "metadata": {"location": "somewhere"}, "photo_url": "http://s3-media4.fl.yelpcdn.com/assets/srv0/yelp_large_assets/3f74899c069c/assets/img/illustrations/mascots/[email protected]"}
  2. The following entities must be populated:
    Rule, MeetingSubscription, DateTimeSubscription

@kentwills
Copy link
Contributor Author

@bxm156 @elg0nz to clarify on the point above, the requirement is only to have a location param because the rule was set to look at location.

This highlights that rules are required. Without them being added to a subscription, the user will not see the subscription (it's a whitelist). Maybe this shouldn't be the default?

@carnalim
Copy link

Looking through this project do you mind adding my email on my profile as well. The directions are not that intuitive, I just picked up google cloud for this project and am combing through any documentation and issues but there is almost nothing to go off of.

@kentwills
Copy link
Contributor Author

@carnalim Added! Yeah this is pretty tough right now because you have to be familiar with the ecosystem and know what the app expects. We just simplified this by one step, removing the need for rules for a subscription. To update the database, everything is done through the datastore in appengine at the moment. I would like to see a script that people can run to get people started.

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

5 participants