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

Report to Odoo16 instead of intermediate postgresql #151

Closed
wants to merge 8 commits into from

Conversation

HappyTetrahedron
Copy link
Contributor

@HappyTetrahedron HappyTetrahedron commented Nov 10, 2023

Summary

This replaces the transformation process: instead of creating interim results to be stored in Postgres, we now create metered billing records that are sent directly to Odoo16. Invoicing is no longer managed by this component.

This is a breaking change. The system is fundamentally reworked.

Checklist

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • Update tests.

@HappyTetrahedron HappyTetrahedron added enhancement New feature or request breaking labels Nov 10, 2023
@HappyTetrahedron HappyTetrahedron requested a review from a team as a code owner November 10, 2023 15:38
@HappyTetrahedron HappyTetrahedron marked this pull request as draft November 10, 2023 15:38
@HappyTetrahedron HappyTetrahedron force-pushed the feat/odoo16 branch 4 times, most recently from f0a7f6a to 6a93bd1 Compare November 13, 2023 14:35
@HappyTetrahedron HappyTetrahedron marked this pull request as ready for review November 13, 2023 14:45
go.mod Outdated Show resolved Hide resolved
pkg/db/seeds/promtest/appuio_cloud_loadbalancer.jsonnet Outdated Show resolved Hide resolved
pkg/odoo/odoo16.go Outdated Show resolved Hide resolved
pkg/odoo/odoo16.go Outdated Show resolved Hide resolved
pkg/report/report.go Outdated Show resolved Hide resolved
pkg/report/report.go Outdated Show resolved Hide resolved
pkg/report/report.go Outdated Show resolved Hide resolved
pkg/report/report.go Outdated Show resolved Hide resolved
Copy link
Contributor

@bastjan bastjan left a comment

Choose a reason for hiding this comment

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

Looks good to go/ good to try now 😊

pkg/report/report.go Outdated Show resolved Hide resolved
pkg/report/report.go Outdated Show resolved Hide resolved
@HappyTetrahedron
Copy link
Contributor Author

Superseded by appuio/appuio-reporting#3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants