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 'Billed' sensors to show actual price per hour #89

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DCSBL
Copy link
Collaborator

@DCSBL DCSBL commented Jul 28, 2023

Closes #88

*_price_including_tax + *_sourcing_markup geeft de prijs per uur aan die gebruikt zou kunnen worden in het dashboard, om de juiste betaalde prijs te krijgen in Home Assistant.

@DCSBL
Copy link
Collaborator Author

DCSBL commented Aug 1, 2023

Is nog draft, als #88 duidelijk is zal ik er is naar kijken :)

@HiDiHo01
Copy link
Contributor

HiDiHo01 commented Aug 1, 2023

Is nog draft, als #88 duidelijk is zal ik er is naar kijken :)

Top
Ik heb ook nog veel wijzigingen maar ik weet niet hoe ik die allemaal voor kan stellen.
Ik ben niet echt thuis in github maar Python begin ik al aardig onder de knie te krijgen.

@DCSBL
Copy link
Collaborator Author

DCSBL commented Aug 1, 2023

In de basis werken veel repo's als volgt: Je idee → Issue → PR → Approval → Merge. Dit niet verplicht maar veel gebruikt.

Waarbij persoon die de PR maakt niet dezelfde hoeft te zijn. Daarnaast kan je ook gewoon in één keer een PR maken. Zorg ervoor dat elke issue/PR gefocust is op één ding en alleen het noodzakelijke aanpast. Bijvoorbeeld; Fix geen typo in file1 als je een bug oplost in file2, dit zijn dan 2 PR's.

Kijk ook eens hier https://developers.home-assistant.io/docs/review-process/#creating-the-perfect-pr, daar heb ik veel aan gehad.

@HiDiHo01
Copy link
Contributor

HiDiHo01 commented Aug 1, 2023

In de basis werken veel repo's als volgt: Je idee → Issue → PR → Approval → Merge. Dit niet verplicht maar veel gebruikt.

Waarbij persoon die de PR maakt niet dezelfde hoeft te zijn. Daarnaast kan je ook gewoon in één keer een PR maken. Zorg ervoor dat elke issue/PR gefocust is op één ding en alleen het noodzakelijke aanpast. Bijvoorbeeld; Fix geen typo in file1 als je een bug oplost in file2, dit zijn dan 2 PR's.

Kijk ook eens hier https://developers.home-assistant.io/docs/review-process/#creating-the-perfect-pr, daar heb ik veel aan gehad.

Dank je, ik zal mij er eens in verdiepen en dat artikel lezen.
Probleem is dat het veel code optimaliseringen zijn. Ik heb een hele lijst prompts voor GPT gemaakt en na een refactor voer ik dan sommige wijzigingen door in mijn code. (ik neem niet zomaar alles over want GPT doet soms ook hele rare dingen.

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.

markt prijzen + BTW + FE markup prijs
2 participants