On Thu, Sep 27, 2018 at 5:04 PM Vincent Massol <[email protected]> wrote:

> Hi devs,
>
> I’ve just had a quick chat with Edy and I found that we had a difference
> of opinion on the Code subspace practice.
>
> On
> https://dev.xwiki.org/xwiki/bin/view/Community/ApplicationDevelopmentBestPractices
> we say:
> "Technical pages must be put in a subspace named Code”
>
> Now Edy says that this should be done only for data-generating apps.
>

This sounds to me like an exception from a rule. Time has taught us that
nothing is unchangeable and we can expect at some point to have some
content in a non-data-generating app.
I would prefer to keep the current rule.


> It’s not my recollection that this rule was only for this case and this is
> what I’d like to discuss here.
>
> For example, I’ve noticed that ActiveInstalls has all technical pages
> under ActiveInstalls, see
>
> https://github.com/xwiki/xwiki-platform/tree/053f0a2757cea18a5916632a58c6046ba61954cd/xwiki-platform-core/xwiki-platform-activeinstalls/xwiki-platform-activeinstalls-server/xwiki-platform-activeinstalls-server-ui/src/main/resources/ActiveInstalls
>
> I would fix it to have only the WebHome remain under ActiveInstalls and
> move all the technical pages under ActiveInstalls.Code.
>
> The only case where it could make sense to not have a Code subspace would
> be when the app has no UI at all. Even in this case, you might argue that
> we should always have a home to provide a description about the content of
> the space.
>
> So right now I’m personally in favor of continuing the rule we defined in
> the best practices:
> "Technical pages must be put in a subspace named Code”
>
> WDYT?
>
> Thanks
> -Vincent
>
> Thanks,
Alex

Reply via email to