terraform-genai-doc-summarization icon indicating copy to clipboard operation
terraform-genai-doc-summarization copied to clipboard

chore!: simplify webhook code

Open davidcavazos opened this issue 10 months ago • 1 comments

Main changes

⚠️ Note: This PR includes breaking changes, but will make the code and maintenance simpler at the long run.

Terraform

  • Simplify terraform to make it easier to maintain
  • Simplify required permissions and roles
  • Include unique prefix on resources to avoid name clashes with other solutions
  • Update terraform package versions
  • Update Makefile to generate metadata
  • Move BigQuery schema to its own file

Webhook code

  • Simplify webhook code
  • Simplify webhook tests
  • Use Document AI instead of Vision API
  • Use Gemini instead of PaLM
  • Improve webhook github actions linter and tests
  • Add Python lint check with ruff
  • Add Python type checking with mypy
  • Upgrade Python dependencies
  • Remove notebook since it's already covered by the Neos tutorial

TO DO before merging

  • [x] [cl/625435521] Update Neos tutorials
  • [x] [pricing-calculator] Update pricing
  • [ ] [planned] Update documentation
  • [x] [same PR] Update architecture diagram
  • [x] [cl/626184064] Update solution metadata in Console
  • [ ] Update the sic-jss tag

davidcavazos avatar Apr 12 '24 00:04 davidcavazos

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB