Skip to content

feat(bounty): add report button that append data to a spreadsheet using Rows API#81

Merged
ricardofelgueiras merged 10 commits intorows:mainfrom
crocodile-bounty-hunter:main
Apr 19, 2024
Merged

feat(bounty): add report button that append data to a spreadsheet using Rows API#81
ricardofelgueiras merged 10 commits intorows:mainfrom
crocodile-bounty-hunter:main

Conversation

@crocodile-bounty-hunter
Copy link
Copy Markdown
Contributor

@crocodile-bounty-hunter crocodile-bounty-hunter commented Apr 16, 2024

Overview

This PR introduces a significant enhancement to the Help state by integrating user feedback and error reporting with Rows spreadsheet submissions, replacing the previous "Talk to Us" feature. It also automates the setup of necessary configurations using GitHub Actions.

Key Changes

1. Action Replacement and Reporting:

  • The "Talk to Us" feature is replaced by an automated submission process to a Rows spreadsheet.
  • For pages without detectable tables, a "Report" button logs details (UTC datetime, URL, domain, browser details, and report type) to the spreadsheet.
  • A "Report" link is persistently displayed on all pages, allowing users to report issues related to table detection or other concerns via a modal with multiple choices.
    2. Configuration and Automation:
  • All necessary Rows API keys are now included in the environment configuration file.
  • A new GitHub Action step has been added to automatically generate this configuration file during the release process, ensuring that all settings are correctly applied without manual intervention.

This update aims to streamline user feedback mechanisms and enhance data capture accuracy for better service improvements.

Comment thread src/utils/rows-api/report.ts Outdated
Comment thread src/utils/rows-api/report.ts Outdated
@crocodile-bounty-hunter crocodile-bounty-hunter changed the title feat(bounty): add report button that append data to a spreadsheet usi… feat(bounty): add report button that append data to a spreadsheet using Rows API Apr 17, 2024
@ricardofelgueiras
Copy link
Copy Markdown
Contributor

image image

@crocodile-bounty-hunter
Copy link
Copy Markdown
Contributor Author

Hey @ricardofelgueiras thanks for you feedback. Regarding the "feedback form".

What do you think about this?

image

@crocodile-bounty-hunter
Copy link
Copy Markdown
Contributor Author

you can test it here.

@ricardofelgueiras
Copy link
Copy Markdown
Contributor

Hey @ricardofelgueiras thanks for you feedback. Regarding the "feedback form".

What do you think about this?

image

It's ok.

@ricardofelgueiras
Copy link
Copy Markdown
Contributor

ricardofelgueiras commented Apr 19, 2024

A few comments on the PR:

  • Add the .env file to the .gitignore
  • Change the deploy action on the step Build chrome extension dist files and start passing the three env variables via secret. On my side I'll make sure the variables are configured on the repository before you merge it.
  • Add a mention to the .env file on the README file explaining why and for what's is being used, and how to set it up when running the extension locally.

Thank you.

@crocodile-bounty-hunter crocodile-bounty-hunter marked this pull request as ready for review April 19, 2024 09:27
@crocodile-bounty-hunter crocodile-bounty-hunter requested a review from a team as a code owner April 19, 2024 09:27
@crocodile-bounty-hunter
Copy link
Copy Markdown
Contributor Author

A few comments on the PR:

  • Add the .env file to the .gitignore
  • Change the deploy action on the step Build chrome extension dist files and start passing the three env variables via secret. On my side I'll make sure the variables are configured on the repository before you merge it.
  • Add a mention to the .env file on the README file explaining why and for what's is being used, and how to set it up when running the extension locally.

Thank you.

Done!

Comment thread src/utils/rows-api/fetch.ts Outdated
Copy link
Copy Markdown
Contributor Author

@crocodile-bounty-hunter crocodile-bounty-hunter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

solve comments from @ricardofelgueiras

@ricardofelgueiras ricardofelgueiras merged commit 578f7c5 into rows:main Apr 19, 2024
dashdash-ci-bot pushed a commit that referenced this pull request Apr 19, 2024
# [1.14.0](v1.13.0...v1.14.0) (2024-04-19)

### Features

* **bounty:** add report button that append data to a spreadsheet using Rows API ([#81](#81)) ([578f7c5](578f7c5))
feep added a commit to feep/X that referenced this pull request Apr 19, 2024
The deleted bounty was (I believe) closed by rows#81.
ricardofelgueiras pushed a commit that referenced this pull request Apr 19, 2024
The deleted bounty was (I believe) closed by #81.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants