Skip to content

Add 'Billed' sensors to show actual price per hour#89

Draft
DCSBL wants to merge 1 commit intomainfrom
DCSBL-patch-3
Draft

Add 'Billed' sensors to show actual price per hour#89
DCSBL wants to merge 1 commit intomainfrom
DCSBL-patch-3

Conversation

@DCSBL
Copy link
Copy Markdown
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
Copy Markdown
Collaborator Author

DCSBL commented Aug 1, 2023

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

@HiDiHo01
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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