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

tms 92: project setup readme #4

Merged
merged 32 commits into from
Mar 20, 2024
Merged

Conversation

tiddoloos
Copy link
Collaborator

Eerste aanzet voor een readme met daarin principes en bronnen voor de woningwaardering berekening, naamconventies en package design.

Copy link
Contributor

@sTomerG sTomerG left a comment

Choose a reason for hiding this comment

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

Ziet er goed uit! Paar kleine suggesties

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
docs/afbeeldingen/project_design.excalidraw Outdated Show resolved Hide resolved
tiddoloos and others added 2 commits March 11, 2024 12:54
Co-authored-by: Tomer Gabay <tomergabay001@gmail.com>
sTomerG
sTomerG previously approved these changes Mar 11, 2024
@benverhees
Copy link
Collaborator

benverhees commented Mar 18, 2024

Ik zou in de voorbeelden en waar we spreken over de definitie van data in CVS begindatum en einddatum gebruiken ipv peildatum. Peildatum wordt gebruikt voor de input, om te bepalen welke waarden/functies we gebruiken op basis van de begindatum en einddatum.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@tiddoloos tiddoloos merged commit ea912f4 into main Mar 20, 2024
3 checks passed
@tiddoloos tiddoloos deleted the feature/TMS-92-project-setup-readme branch April 2, 2024 12:53
@benverhees benverhees added the enhancement New feature or request label May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants