« Back Issue #428
  • 2015-04-05 18:04 Reported by pooja
  • 2015-12-26 18:02 Marked fixed
Status: Marked fixed
Priority: Medium

Implement the User Interface for news updater and optimize it

pooja 2015-04-05 18:04

This change will let one control the presence of media_updater on Name server or rest of the machines in the cluster. There is a new button called as 'Independent Media Updater'. Clicking on this button would flip the Field_value of Present_On_Media_Updater fied from database and refresh the manage machines page based on the current value of the field Present_On_Media_Updater in table MEDIA_UPDATER_STATUS. Everything mentioned above will also be done whenever the manage machines page gets updated in every 20 seconds.

pooja 2015-04-05 18:10

Hello Dr. Pollett,

I have raised a new ticket for my new patch. Also, in relationships field I have marked it as the child of the last issue 0000161. I will update this issue with patch file for any suggestions by tonight.

Thanks, Pooja

administrator 2015-04-05 18:58

Okay. I'll take a look at it tomorrow morning then.

Best, Chris

pooja 2015-04-06 19:58

Hello Dr. Pollett,

Please have a look at the patch. Let me know your suggestions.

Also, I am not sure how to give database changes as they wont be in this patch.So I have uploaded another text file named database_changes having the create and insert statements for table MEDIA_UPDATER_STATUS that I had created.

Let me know if you need anything else.

pooja 2015-04-10 14:31

Hello Dr. Pollett,

Please check the updated patch file.

Thanks, Pooja

pooja 2015-04-10 17:18

Hello Dr.Pollett,

I have uploaded the latest patch. Please have a look and let me know.

Thanks, Pooja

administrator 2015-04-11 09:16

Hey Pooja,

I've applied this patch, but not pushed it live. It still has some minor issues like a stray 3 at the top of one of the view files and you didn't localize one of the strings you echo'd in a controller. I am going to try to fix these myself over the next day or so. I'll let you know when I succeed or if you need to do more on this.

Best, Chris

pooja 2015-04-11 12:04

Sure Dr. Pollett.

I skipped those things by mistake. I can fix these issues and upload another patch as it wont take much time for me to do this. Let me know when you want me to that.

Thanks, Pooja

administrator 2015-04-12 20:30

Thanks for your patch Pooja! I went through your code and modified it. The result is now pushed live and you can get it to see what I changed. One thing that you should watch out for that could affect future patches is what types you give fields in create statements. I think the TEXT type is supported by SQLITE but not by other DBMS's. I am leaving this issue open for now as I have one or two questions still for tomorrow.

pooja 2015-04-13 18:46

Note to myself:

Make sure Name server is available in both the modes(Name server as well as Distributor). UI change. Include this in the next patch.

administrator 2015-12-26 18:02

This has been fixed since some time in mid-2105 -- just getting around to closing it. Thanks for your work Pooja!

pooja 2015-12-26 18:02

Attachments:

my.patch

my2.patch

X