Closed GoogleCodeExporter closed 9 years ago
Sorry Amr, I want to ask about this. do you mean change the status in general
to anything?
In the success, make sure that the user is notified with the change in his
status which is done by others ;)
What kind of changes that can be made after the deadline had passed?
Peace
Original comment by felsayed...@gmail.com
on 16 Jun 2010 at 9:50
I mean the meeting status(sorry for not clarifying that in the story)...
attended or not... we added this because for example a user that entered his
status as attended and did not attend for any reason so the changes of the
status will be either attended or not attended.
oh yeah Okay a mail will be sent thanks for that :D
This change can only be done by a user who is having the role of setting
attendance.
Is that's okay ?
Thanks :)
Original comment by amoro...@gmail.com
on 16 Jun 2010 at 10:04
Yes sounds good.
So, there is a permission for changing attendance (of others) and this one is
a powerful one which can work even after the meeting has closed or at anytime
And there is another persmission given to the user to change (his own)
attendance which should be limited since he shouldn't say that he attended
after the meeting has expired or it might be misused?
It is an interesting debatable feature :) And it will clarify with a demo. Let
me know whenver you have anything working. I am the product owner now and I
don't have to wait till the demo. I can see partial work ;)
Shokran Amrotto :)
Original comment by felsayed...@gmail.com
on 17 Jun 2010 at 8:28
actually I though it (the permission of changing others attendance) will be
working after the meeting time is finished (closed) not before to give normal
users the chance to confirm/decline the invitation.
And for Normal users as you said they will be able to put their attendance
until the meeting time start (its done in sprint 2 already :D )
Welcome Dr.Fatma :D
Original comment by amoro...@gmail.com
on 17 Jun 2010 at 11:05
Hey Dr.Fatma..
I worked on the story and its 99% working now I just want a feedback on it like
the UI and If you want any other thing in it...
You can access the story by clicking on any past meeting then set attendance
(only work with users who have setMeetingAttendance permission) .
Original comment by amoro...@gmail.com
on 21 Jun 2010 at 2:37
Amrotto,
I was able to see set attendance even when the meeting is not closed. I mean it
will end on 2nd of july.
I thought you meant I can only do that after it is closed.
Also you have the three meetings listed under past meetings, do you mean past
and current??
Also, about the UI for the set attendance form. It is nice to make a difference
in color between instructions and the actual form or meat of the work.
The radio buttons definitely have to be aligned for visibility. As if there
they are in a table.
also, the alerts and messages should be in different color and smaller fonts.
Then, after I successfuly enter the reason, you can just hide the textbox and
the button.
Put an astrix for the text box of the reason.
Nice work amrotto :))
Original comment by felsayed...@gmail.com
on 21 Jun 2010 at 9:48
Hey Dr. Fatma...
well we (C2) agreed that the meeting is "past" just when the start time passed
and you can not send any new invites too... so I worked on it that you can set
the attendance of the invited users just when the meeting start.
The UI is now changed and the alerts and messages are handled as you said.
So can you please take a look at it and see if this what you was expecting :D..
thanks a lot..
Original comment by amoro...@gmail.com
on 21 Jun 2010 at 10:53
is it possible to allow invites as long as the meeting time is not over?
Imagine we are having a meeting now and you reminded me that someone show be
invited and most probably, he will be able to see it and get in before the
meeting expires?
So, why close invites before the meeting expires??
Sorry about the delay but i didn't test your work yet. I will do it today when
I go home. I had internet issues yesterday.
Thanks amorotto :))
Original comment by felsayed...@gmail.com
on 22 Jun 2010 at 2:17
Looks good amrotto :)
Good job
I understand that the guy whose status changed is notified, right?
Thank you
Original comment by felsayed...@gmail.com
on 22 Jun 2010 at 7:19
Original comment by amoro...@gmail.com
on 26 Jun 2010 at 6:45
amr ,from where can i change the status of the attendees???
ana 3amlt 2 meeting & they are past already ,bas msh 3aref a3'ayer elstatus mnen
also this error popped when i did try to confirm the meeting through the link
in the notifications .
bas lma 3mlt confirmation mn my invitation ,worked fine
ana msh 3aref dah tab3ak wla la2??,bas 2olt a2olak :D
NullPointerException occured : Try to read status on null object
models.MeetingAttendance (controllers.MeetingAttendances.confirm, ligne 32)
In /app/controllers/MeetingAttendances.java (around line 32)
28:
29:
public static void confirm( String meetingHash )
30:
{
31:
MeetingAttendance attendance = MeetingAttendance.find( "byMeetingHash", meetingHash ).first();
32:
String status = attendance.status;
33:
Date currentDate = new Date();
34:
Date tempStart = new Date( attendance.meeting.endTime );
35:
boolean notYet = tempStart.after( currentDate );
36:
boolean setbefore = false;
37:
User user = Security.getConnected();
38:
boolean isUser = false;
Original comment by Moumen.Elteir@gmail.com
on 28 Jun 2010 at 9:01
Amr, could you also please use the note tag for the notes in this page:
http://localhost:9000/meetingattendances/setattendance?meetingID=4
Love your work btw, spectacular! :)
Original comment by mahmoudsakr87
on 2 Jul 2010 at 4:37
Original comment by amoro...@gmail.com
on 25 Jul 2010 at 5:22
Working like a charm :)
Original comment by galalaly28@gmail.com
on 25 Jul 2010 at 11:43
Original issue reported on code.google.com by
amoro...@gmail.com
on 16 Jun 2010 at 8:34