-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add Portal rating #475
Comments
@ebma @Sharqiewicz anything unclear about the above? is it possible to capture the data on excel sheet? Marked it as high priority as it is one, once estimated i would request to work on this asap. |
Hey team! Please add your planning poker estimate with Zenhub @ebma @Sharqiewicz |
Isn't this a bit too pushy? I would be annoyed if the app was asking me for feedback all the time, even if I manually close the dialog. I think if a user closes the dialog and actively decides not to provide feedback, we should also not prompt again for at least a week, or maybe treat it like the default case of 15 days.
A database is definitely overkill for this so let's rather try to store it differently. While I've never tried it myself, I guess we can send that data to a Google Form. |
@Sharqiewicz please add you estimates |
Fair point, made changes in the ticket |
Please add your planning poker estimate with Zenhub @TorstenStueber |
We would need some backend for this, I think we should not directly push to the Google Form from the frontend, otherwise we expose the credentials. |
I think we can use Netlify Functions along with Google Sheets. It won't expose credentials. WDYT? @TorstenStueber @ebma |
Since Netlify functions use AWS lambda internally, we can also just create a lambda function for this right away. We can then store the credentials in AWS secret manager for example. Creating an extra server that runs 24/7 just for this might be overkill. |
Yes, Lambda or Netlify functions sounds good. This can be setup and managed by DevOps. |
The code can be copied from Vortex |
Yes, let's re-use the great component you already created in pendulum-chain/vortex#182. |
As a user i should be able to rate the portal
Acceptance criteria
Wireframe
https://www.figma.com/proto/5up5sghcttBogehUq4255i/App?node-id=2849-5156&t=t26OmPb2nSn7Axcj-0&scaling=scale-down&page-id=136%3A114&starting-point-node-id=2849%3A5156
The text was updated successfully, but these errors were encountered: