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

Missing class as fields of loan #11

Open
laney0808 opened this issue Apr 19, 2024 · 1 comment
Open

Missing class as fields of loan #11

laney0808 opened this issue Apr 19, 2024 · 1 comment

Comments

@laney0808
Copy link
Owner

laney0808 commented Apr 19, 2024

In Developer Guide's class diagram, the fields of loan (value, start date, end date) are missing.

Screenshot 2024-04-19 at 17.39.33.png

@nus-pe-bot
Copy link

nus-pe-bot commented Apr 22, 2024

Team's Response

value: BigDecimal
start, end: Date

These are default Java classes and not defined by the developers. Moreover, these fields/attributes are not crucial in representing the Model class and can be omitted. (Source: CS2103T course website)
Screenshot 2024-04-21 at 5.20.55 PM.png

As these fields are not integral to understanding how the "Model" component operates, we have left these details out for more readability.

Bug is rejected since this is expected behavior.

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: [replace this with your explanation]


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants