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

Add Export Functionality #162

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

flowirtz
Copy link

@flowirtz flowirtz commented Feb 25, 2022

  • Add button component
  • Implement CRE page
  • Implement standards page
  • Implement generic search page

Fixes #161

https://www.loom.com/share/67d67449e98649dc9fffe0f180d5dee8

@flowirtz flowirtz marked this pull request as ready for review February 25, 2022 11:42
@flowirtz flowirtz changed the title [WIP] Add Export Functionality Add Export Functionality Feb 25, 2022
@northdpole northdpole self-requested a review April 28, 2022 01:46
Copy link
Collaborator

@northdpole northdpole left a comment

Choose a reason for hiding this comment

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

the pattern so far has been to make actions into query strings, e.g. ?export=true
this is to prevent accidentally exporting things when you call
node/US_CRYPTO_EXPORT/foo

i'll fix this in the next few days
other than that it's good work, thank you

@northdpole northdpole self-requested a review April 28, 2022 02:17
@northdpole
Copy link
Collaborator

TODO: @northdpole , add documentation on how to setup a gspread oauth token so that the backend can generate the spreadsheet.

Copy link
Collaborator

@northdpole northdpole left a comment

Choose a reason for hiding this comment

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

This can easily be abused by bots, can you please

  • make this feature optional (hide it behind an environment variable when building the frontend ?
  • add a captcha challenge for clicking the button

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.

Add export functionality
2 participants