Skip to content

207 assign1 toast#25

Open
ToastedToast00 wants to merge 2 commits intomainfrom
207Assign1-Toast
Open

207 assign1 toast#25
ToastedToast00 wants to merge 2 commits intomainfrom
207Assign1-Toast

Conversation

@ToastedToast00
Copy link
Contributor

This is for the 207 assignment 1.

@SuperGamer001 SuperGamer001 added the assignment Issue/PR is for a school assignment label Jan 29, 2026
Copy link

@rachelkingCPTC rachelkingCPTC left a comment

Choose a reason for hiding this comment

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

Just have a question for you -- educate me if I am misunderstanding!

Comment on lines +4 to +7
- App
- verbs: load, start, close, refresh
- Page
- verbs: display, transition, update

Choose a reason for hiding this comment

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

Admittedly, I am very much not a front end developer. Would the concepts of "App" and "Page" be entities that are handled by your framework? I'm questioning if these more belong in the System/Technical category.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

App and page are usually considered technical UI constructs rather than domain entities, because they only exist due to the framework and not the problem space itself.

That's why I placed them in the System/Technical category, they're important for implementation, but they don't represent real domain concepts the system is modeling.

Choose a reason for hiding this comment

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

I see them listed under the Entity category -- they can be removed from there, and their skeleton classes can be removed. It's okay in this analysis if you don't find any Entity level classes. You have a couple of roles; those will end up being classes, probably. Otherwise, good work

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

Labels

assignment Issue/PR is for a school assignment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants