xwikisas / application-meeting

Application to handle meetings
0 stars 7 forks source link

XPoll for a Meeting should be nested in the same space #30

Closed oanalavinia closed 6 years ago

oanalavinia commented 6 years ago

https://jira.xwiki.org/browse/XAMEETINGS-135

oanalavinia commented 6 years ago

I discussed with Silvia and Cati and for motives of consistency the Poll should remain in the XPoll space. There could be problems when XPoll app is uninstalled because this will delete only polls from XPoll space, leaving those from Meeting space there. Also, even this functionality does not exists yet, maybe for future versions there could be made a reference to an existing poll and in this case the parent will be XPoll.

mflorea commented 6 years ago

There could be problems when XPoll app is uninstalled because this will delete only polls from XPoll space, leaving those from Meeting space there.

The polls (wiki pages) are not removed when you uninstall the Poll application. Only the application code pages are removed.

evalica commented 6 years ago

So, for this particular use case Poll/Meeting, I don't think is necessary to break the default behavior.

We should check if the created poll was deleted meanwhile and mark the Meeting accordingly. Maybe preserving the broken link, so that the Admin could restore the Poll in case of need.

evalica commented 6 years ago

In case the XPoll app is uninstalled, even if the Poll page will be preserved under the Meeting, it will lack the display sheet, so it will look ugly.

mflorea commented 6 years ago

Having the Polls into the XPoll space also helps in terms of discovery

The live table from the Poll application home page is most probably listing polls located anywhere (that's the default behavior when you create an application with AWM), so there's no discoverability issue I think.

oanalavinia commented 6 years ago

I tried and indeed the polls listed in the live table are from anywhere

CostiChD commented 6 years ago

How about adding a column in the livetable(from XPoll/WebHome) to show the connection between the poll and the Meeting App(maybe something like a "category") ? Now we see the polls listed, but we dont know exactly what each poll is about(unless the name gives enough information).

lucaa commented 6 years ago

@oanalavinia what's the conclusion of this issue, actually? it's closed but I don't understand if it was done or not or won't be done at all? I see no commit on it, so I guess it wasn't done?

oanalavinia commented 6 years ago

@lucaa I closed this issue because it didn't had enough votes for it to be done