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 button #52

Merged
merged 4 commits into from
Mar 2, 2024
Merged

add button #52

merged 4 commits into from
Mar 2, 2024

Conversation

haysho2260
Copy link
Contributor

Developer: {Hayley Chang}

Closes #{45}

Pull Request Summary

{Add button for webscraper}

Modifications

{add button and description to run webscraper}

Testing Considerations

{list out what you have tested and what the reviewer should verify}

Pull Request Checklist

  • Code is neat, readable, and works
  • Comments are appropriate
  • The commit messages follows our guidelines
  • The developer name is specified
  • The summary is completed
  • Assign reviewers

Screenshots/Screencast

{put screenshots of your change, or even better a screencast displaying the functionality}

@haysho2260 haysho2260 linked an issue Feb 20, 2024 that may be closed by this pull request
2 tasks
Copy link

vercel bot commented Feb 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ecologistics-web-scraper ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 2, 2024 9:53pm

Copy link
Collaborator

@ryanchansf ryanchansf left a comment

Choose a reason for hiding this comment

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

See comments! Just a minor fix

return (
<main>
<h1>Home</h1>
<ProposalTable />
Copy link
Collaborator

Choose a reason for hiding this comment

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

The button looks good! But we don't want to get rid of the proposal table just yet. When we eventually link the table to the backend, we'll have some conditional rendering to have it appear only after having clicked the button to run the webscraper. But for now, we can just leave the table as is.

@ryanchansf ryanchansf merged commit 609cf21 into develop Mar 2, 2024
4 checks passed
@ryanchansf ryanchansf deleted the 45-button-to-run-web-scraper branch March 2, 2024 21:53
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.

Button to run web scraper
2 participants