-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Include peer review to app #1559
Comments
I would be in favor of releasing the feature without spending time polishing it because:
But I believe we must add a mandatory (shown every time, no way to "skip in the future") one-page long text explanation that details:
|
@neslihanturan By the way, what branch is it in? Would you mind sharing the URL? Thanks :-) |
I agree with the steps mentioned by @neslihanturan and @nicolas-raoul - preferably we will polish this a bit before incorporating it in our app, and we need the text explanation. How about we include the polishing as part of our PG 2019 plans? Also, actually, now that we have the Browse feature... can we reuse that for this purpose? |
I believe this is done? :-) |
Summary:
As you know, peer review #780 is roughly working. This issue will contain required steps to include it to our master codebase.
What do you suggest?
The text was updated successfully, but these errors were encountered: