RDN-4541
1

Settings Calendar View

Scott on 3/2/2016 5:06 AM

Scott on 5/4/2016 4:38 PM

%
(hrs)
Logged

0 (hrs)


 Description

Calendar Settings is broken.  Please make sure to fix it.  It has worked in the past, but for some reason has stopped working.

  1. Login to RDN.League.
  2. Go to Team/League > Calendar.  
    1. Click on Large or List.
    2. As you can see, this shows the calendar in two ways.
    3. There is a setting to say which one is default, but ha stopped working.
  3. Go to Profile > Settings
  4. Calendar Settings.
  5. Change the List View to Large Calendar View
    1. It should automatically save, but it doesn't.
    2. Please fix this again.

 

 

I can't get the calendar settings to change to default on large. I check the box and nothing happens.  There isn't a "save settings" button.  Am I doing something wrong?

Skip Navigation Links
 

Vasyl Thursday, April 07, 2016 5:10 PM #

@Scott, 

please review pull request here: https://bitbucket.org/rdnation/rdnation/pull-requests/103/rdn-4541/diff

Thanks.

 

Scott Thursday, April 07, 2016 2:10 AM #

looks good.  

Please add that mobile notification code back please.

https://bitbucket.org/rdnation/rdnation/pull-requests/103/rdn-4541/diff

 

Vasyl Wednesday, April 06, 2016 10:19 PM #

@Scott, please review pull request https://bitbucket.org/rdnation/rdnation/pull-requests/103/rdn-4541/diff

Thanks.

 

Scott Monday, March 28, 2016 7:44 PM #

Vasyl,

Please pick up where Arnab left off.

https://bitbucket.org/rdnation/rdnation/pull-requests/103/rdn-4541/diff

 

Scott Saturday, March 19, 2016 2:23 AM #

please fix the merge conflicts..

https://bitbucket.org/rdnation/rdnation/pull-requests/103/rdn-4541/diff

 

Arnab Friday, March 18, 2016 8:17 PM #

Please check calendar and let me know if any issue.

 

Scott Tuesday, March 15, 2016 1:47 AM #

Arnab, Moving forward, please do NOT commit and push two tasks to the same branch.

I CANNOT merge this branch until the comments I have made are fixed for this particular branch.

Please review and update: https://bitbucket.org/rdnation/rdnation/pull-requests/103/rdn-4541/diff

 

Arnab Monday, March 14, 2016 7:22 PM #

I have updated code to show the success message as you wanted.
And to solve the other issue like when you change dropdown then you still get the previous calendar view.

 

Scott Friday, March 04, 2016 8:10 PM #

When I reload the page, I don't see a change in the drop down.  The drop down should be selected for what is currently selected.  

Please also add a ".notify event that says "Calendar Setting Changed".  You can find the notify event in main.js and see how to use it.

 

Arnab Thursday, March 03, 2016 8:56 PM #

Hi,

I checked this and it seems fine. 
The only issue I see is that when you change calendar view from Profile > Settings then it saves caledar view using Ajax and whole page does not refresh so the link in Team > Calendar contains
link of calendar view which was saved before changing view setting using Profile > Settings

To take this change into effect immediately we need to refreash Settings page immediately after we see Saved notify message, so that link in header of page can be updated immediately.

Let me know your thoughts.


Powered by BugNET 1.6.339.0