-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The default sort of the meetings should display most recent meetings at the top #41
Comments
*the meeting on the first page are displayed in asc order, filter by the current date and the current date plus one year. *also, the user can see all the meetings created.
* The meeting on the first page are displayed in asc order, filter by the current date and the current date plus one year. * Also, the user can see all the meetings created. * Code refactory. Removed unnecessary variable and duplicate code. * Changed double quotes to single quotes * Removed redundant option from livetable
So, let me understand @ignatescustefan , the fix for this issue reverted partially the fix for #36 , it changed the order of meetings to ascending by startDate? As @acotiuga explained on #36 , we need to properly define what "most recent" means, otherwise we'll do these kind of reverts all the time. Otherwise put, the main usecase for the meeting application should be: many meetings in the past and rather little meetings in the future, even if, during a test, this usecase may not be so obvious... |
So, I re-read the commits and the code more attentively, and now I understand better: there is a new behaviour that was invented in order to somewhat satisfy this strange understanding of "recent" as a thing in the future. Note that it's best to explain functional behviour changes on the issue rather than in the pull request. As a user, I don't read pull requests but I do read issues. So, if I understand correctly, it now works like this:
Given this:
and the fact that:
I consider this ticket to be uselessly complicating the functioning of this application. |
My interpretation of “most recent” was also as reverse chronological order. It’s how I initially understood the ticket, thinking the attached image was the expected behavior. (BTW, it would be useful for attached images to explain whether they describe the current or the expected state.) I also agree that the application is usually used mainly to log info about past meetings. If the app were used mostly to plan and have the meetings, it would make sense to only display current and future events and archive past meetings. But the main purpose of the app is to track past events and store old notes for future reference.
|
@lucaa @silviamacovei I don't agree with you. From my point of view seeing an overview of the next planned meetings on the application home page is more useful (and easier to understand) than seeing all the meetings (from past and future). Being able to access all the meetings is obviously important, which is why we added a link. Ideally I would have liked to have the live table filters prefilled so that it was more clear that there is a filter applied on the listed meetings (the next planned meetings filter) but I think some live table limitation prevented us to do this. Now let's see what you propose:
|
No, I am saying that having by default access to all the meetings that have happened in the past and have written meeting notes is more important than seeing the planned meetings.
Ilie's is not a real-life usecase, it's a test usecase, he only has future meetings in his example, no past meeting notes stored. Since time has this funny habit of passing, if there is one thing that we are sure of is that everything will become past at some point. So the usecase where there is no past is, to me, an artificial one (that can happen in a test setup or in an initial phase of a real-life usecase but not on the long term). |
Hi! I understand the rationale described by Anca and the one presented by Marius. |
@lucaa the application is not called Meeting Notes but Meeting and it's description says:
On its home page it says:
Organizing a meeting is not only about keeping meeting notes. Meeting notes are obviously very important, but preparing the meeting, agreeing on a meeting agenda, adding links to resources, inviting people are also important. The point of listing the next planned meetings on the home page is to help the users prepare for their meetings (check the meeting agenda, see who confirmed the invitation). |
@mflorea I see. Then there is an important problem of app definition and usage according to this definition, which, as Ilie also said, would need to be fixed, in one direction or the other.
Of course it's a valid usecase around meetings and a very important one, it's just, from my pov, a market where this application cannot necessarily stand a fight, whereas the knowledge management of the notes is one. |
The description we currently have on the store was mostly reused from the initial description on exo. While there have been many bug fixes and improvements in the paid app, due higher demand on other apps, we haven’t spent time on trying to redefine the main use cases for this one. Right now the app does a bit of everything and is based on the 2012 version. Back then it made sense to have a full meeting lifecycle. Nowadays, meeting management and planning continue to be valid use cases, but the reality is most users plan their meetings through their Google Calendar, Outlook etc. And while these tools handle the planning and actual meeting parts very well, they don’t offer an easy way to store and search through meeting notes. This is why, on deeper consideration, I agree that while all use cases are valid, this is the main use case where we can bring the most value. I think we should update the documentation to reflect this, if we'll all agree. Of course, one could also argue that we could keep improving it to get the app to satisfy all of the use cases really well. However, because people sync with their existing calendars, we haven’t seen demand for the planning part. And there are many other cool things we could be working on, apps wise, that could bring extra value to our users, value that they can't easily get in other places. |
To be clear, by planning a meeting I don't understand just setting a date for the meeting and inviting the people. I wouldn't keep the meeting agenda and resources on Google Calendar or Outlook but closer to the meeting notes, so that for instance I can check afterwards easily if all the topics from the agenda were covered. In any case, my proposal is to have two tabs (as in the page index) on the application home page:
I'm fine to select the first tab, Meeting Notes, by default, based on @lucaa and @silviamacovei feedback. |
@mflorea very good idea the tabs, it will do the job perfectly, easy to access for users and clear (no long text to read to understand). While at it, we could also add the calendar view as a tab, it would really make sense, rather than a link like it is today. |
+1 |
The three tabs sound good! For the proposed naming of the first two tabs I would go for:
|
STEPS TO REPRODUCE
EXPECTED RESULTS
The meetings should be sorted starting with the most recent meetings first (at the top), as it's a common use case, indicated also in #36.
ACTUAL RESULTS
The meetings are sorted with the most recent meetings at the bottom.
Environment: Windows 10 Pro 64-bit, Chrome 70, using a local instance of XWiki 10.9 on Oracle 11.2.
The text was updated successfully, but these errors were encountered: