Dormbase

Merge AbstractRoom→Room and AbstractResident→Resident

Bug #1140478 reported by Luke Faraone
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Dormbase
Fix Committed
High
Tom Boning
Dormbase march2-hackathon

Bug Description

The concrete class names are overly generic, and there are ways to represent site-specific data when the need arises. I don't think there's huge value in having this complexity at the present time.

Tags: models
Revision history for this message
Tom Boning (tboning) wrote :

I'm going to look into merging the classes.

Changed in dormbase:
assignee: nobody → Tom Boning (tboning)
Revision history for this message
Tom Boning (tboning) wrote :

I don't think merging the classes will break anything. Most references are direct to the subclass.

Changed in dormbase:
status: Triaged → In Progress
Revision history for this message
Tom Boning (tboning) wrote :

Pull Request Sent.

Changed in dormbase:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.