- AppBuilder
- does that mean we support only content app descriptors from AppBuilder?
- Why is AppBuilder part of contacts app? Is it only ContactsAppBuilder rather then generic one? If so, it needs to be renamed.
- ContentAppBuilder
- what is the difference between AppBuilder and ContentAppBuilder now that both use Content?
- Why is it in contacts package? If it is not generic, it needs to be renamed to ContactContentAppBuilder
- SubAppBuilder
- it is now just ContentSubAppBuilder and if it is not generic it needs to be renamed to ContactsSubAppBuilder. If it is generic, in needs to be moved out of contacts app module.
- AppContext
- This kind of javadoc is totally useless. If you don't plan to write any javadoc, don't leave this in the code.
|