-
Notifications
You must be signed in to change notification settings - Fork 7.3k
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
Realistic data for RealWorld? #77
Comments
Love this idea. @apai4 and I were originally kicking around creating profiles for authors from the 1900's and earlier (to avoid copyright restrictions) and post excerpts from their books/essays/etc at random intervals. Would be cool/hilarious to see Ben Franklin, Sun Tzu, etc posting articles on conduit :) |
That actually sounds pretty cool. |
But how would this be helpful when so many users are constantly testing the demo. Unless access to create articles are restricted then the demo would soon be filled with user based content displaying on the front. Like @EricSimons mentions, inserting at random intervals might help, but that would require another setup. It might be a good idea to create a dataset for new users to import to their database to get started. This might help with testing as well. |
Good dataset could be description of Realword project of some sort: articles, comments, etc. |
Ok. I think I've got an interesting solution to this one.
Given (1) and (2), here is what I propose:
Specifically, this means that users (both logged in and logged out) should not see posts made by other users. By allowing logged in users to see their own posts we solve the above problem of hiding posts making the demo useless. By not allowing users to see posts by other users, we also solve the problem of surfacing only good content that accurately reflects the abilities of Conduit. Another benefit is that this solves the pagination problem we ran into in the past where we were showing 100+ pages of content. Thoughts? EDIT: This will require a fork of whatever front-end/back-end that is currently running the demo. These changes should not be made in the public codebases. |
^ love this idea @BRWR! How far did you get on Sat with the Rails fork? |
So which version of the current backend are we modding to get this rolling? I assume it's mostly gonna be node. I can probably get this done myself if you're fine with having the laravel version of api running in prod. |
The demo is running a version of the Rails API.
…On Tue, May 9, 2017 at 12:44 PM, Sandeesh ***@***.***> wrote:
So which version of the current backend are we modding to get this
rolling? I assume it's mostly gonna be node. I can probably get this done
myself if you're fine with having the laravel version of api running in
prod.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#77 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AD5-rBqD4XVn4Yvl5F6nQpi2s1luYZG4ks5r4MI1gaJpZM4NMYpF>
.
|
If this is going to be done, there should be an API-based data generator in this repo I think. Faker is a well known ruby solution: https://github.com/faker-ruby/faker The Js port (preferred for inclusion in this repo): https://github.com/Marak/faker.js On a related note, I'm using a local non-realistic but predictable direct-to-Sequelize-db version which is a must for the development:
But it is sequelize specific. Something like that for API would be great too though. |
I wonder if it's possible/feasible/helpful to fill the production db with more realistic data from some open data sets that might be floating around. It would look better in FE demos than "aaaaa", "sample title" etc... BE repos could also use it for testing their dbs. Just a thought.
The text was updated successfully, but these errors were encountered: