adding "notify" attribute to GenericTask, with amendments to ArgDialog`s UI to set the value

argdialog.ui

@@ -26,7 +26,7 @@ background-color: rgb(0, 43, 54);</string>
     <rect>
      <x>10</x>
      <y>10</y>
-     <width>948</width>
+     <width>946</width>
      <height>891</height>
     </rect>
    </property>
@@ -52,7 +52,7 @@ color:  rgb(255, 85, 0);
      </widget>
     </item>
     <item>
-     <layout class="QHBoxLayout" name="horizontalLayout" stretch="0,0,0,0">
+     <layout class="QHBoxLayout" name="ahorizontalLayout" stretch="0,0,0,0">
       <item>
        <widget class="QLabel" name="label_2">
         <property name="sizePolicy">
@@ -156,7 +156,7 @@ font-weight: 700;
      </layout>
     </item>
     <item>
-     <layout class="QHBoxLayout" name="horizontalLayout_2">
+     <layout class="QHBoxLayout" name="bhorizontalLayout_2">
       <item>
        <widget class="QLabel" name="label_3">
         <property name="sizePolicy">
@@ -358,6 +358,71 @@ font-size: 16pt;</string>
       </item>
      </layout>
     </item>
+    <item>
+     <layout class="QHBoxLayout" name="zhorizontalLayout_8">
+      <item>
+       <widget class="QLabel" name="notificationLabel">
+        <property name="sizePolicy">
+         <sizepolicy hsizetype="MinimumExpanding" vsizetype="Preferred">
+          <horstretch>0</horstretch>
+          <verstretch>0</verstretch>
+         </sizepolicy>
+        </property>
+        <property name="minimumSize">
+         <size>
+          <width>196</width>
+          <height>0</height>
+         </size>
+        </property>
+        <property name="maximumSize">
+         <size>
+          <width>196</width>
+          <height>16777215</height>
+         </size>
+        </property>
+        <property name="styleSheet">
+         <string notr="true">font: 75 11pt &quot;Noto Sans&quot;;
+color: rgb(131, 148, 150);
+font-weight: 700;
+ qproperty-alignment: 'AlignHCenter';</string>
+        </property>
+        <property name="text">
+         <string>Email notification</string>
+        </property>
+        <property name="alignment">
+         <set>Qt::AlignCenter</set>
+        </property>
+       </widget>
+      </item>
+      <item>
+       <widget class="QComboBox" name="notification">
+        <property name="sizePolicy">
+         <sizepolicy hsizetype="MinimumExpanding" vsizetype="Fixed">
+          <horstretch>0</horstretch>
+          <verstretch>0</verstretch>
+         </sizepolicy>
+        </property>
+        <property name="styleSheet">
+         <string notr="true">font: 87 11pt &quot;Noto Sans&quot;;
+background-color: #2f535f;
+color: rgb(131, 148, 150);
+font-weight: 700;
+</string>
+        </property>
+        <item>
+         <property name="text">
+          <string>No</string>
+         </property>
+        </item>
+        <item>
+         <property name="text">
+          <string>Yes</string>
+         </property>
+        </item>
+       </widget>
+      </item>
+     </layout>
+    </item>
     <item>
      <layout class="QHBoxLayout" name="horizontalLayout_7">
       <item>
@@ -433,19 +498,6 @@ font-weight: 700;
       </item>
      </layout>
     </item>
-    <item>
-     <widget class="QLabel" name="label">
-      <property name="styleSheet">
-       <string notr="true">font: 75 11pt &quot;Noto Sans&quot;;
-color: rgb(131, 148, 150);
-font-weight: 700;
-padding-left: 8px;</string>
-      </property>
-      <property name="text">
-       <string>Items to send</string>
-      </property>
-     </widget>
-    </item>
     <item>
      <layout class="QHBoxLayout" name="horizontalLayout_6" stretch="0">
       <item>

headers/generictask.fbs

@@ -10,6 +10,7 @@ table GenericTask {
   header: string;
   user: string;
   recurring: int;
+  notify: bool;
 }
 
 root_type GenericTask;

headers/generictask_generated.h

@@ -22,7 +22,8 @@ struct GenericTask FLATBUFFERS_FINAL_CLASS : private flatbuffers::Table {
     VT_MASK = 14,
     VT_HEADER = 16,
     VT_USER = 18,
-    VT_RECURRING = 20
+    VT_RECURRING = 20,
+    VT_NOTIFY = 22
   };
   int32_t id() const {
     return GetField<int32_t>(VT_ID, 0);
@@ -51,6 +52,9 @@ struct GenericTask FLATBUFFERS_FINAL_CLASS : private flatbuffers::Table {
   int32_t recurring() const {
     return GetField<int32_t>(VT_RECURRING, 0);
   }
+  bool notify() const {
+    return GetField<uint8_t>(VT_NOTIFY, 0) != 0;
+  }
   bool Verify(flatbuffers::Verifier &verifier) const {
     return VerifyTableStart(verifier) &&
            VerifyField<int32_t>(verifier, VT_ID) &&
@@ -67,6 +71,7 @@ struct GenericTask FLATBUFFERS_FINAL_CLASS : private flatbuffers::Table {
            VerifyOffset(verifier, VT_USER) &&
            verifier.VerifyString(user()) &&
            VerifyField<int32_t>(verifier, VT_RECURRING) &&
+           VerifyField<uint8_t>(verifier, VT_NOTIFY) &&
            verifier.EndTable();
   }
 };
@@ -102,6 +107,9 @@ struct GenericTaskBuilder {
   void add_recurring(int32_t recurring) {
     fbb_.AddElement<int32_t>(GenericTask::VT_RECURRING, recurring, 0);
   }
+  void add_notify(bool notify) {
+    fbb_.AddElement<uint8_t>(GenericTask::VT_NOTIFY, static_cast<uint8_t>(notify), 0);
+  }
   explicit GenericTaskBuilder(flatbuffers::FlatBufferBuilder &_fbb)
         : fbb_(_fbb) {
     start_ = fbb_.StartTable();
@@ -124,7 +132,8 @@ inline flatbuffers::Offset<GenericTask> CreateGenericTask(
     int32_t mask = 0,
     flatbuffers::Offset<flatbuffers::String> header = 0,
     flatbuffers::Offset<flatbuffers::String> user = 0,
-    int32_t recurring = 0) {
+    int32_t recurring = 0,
+    bool notify = false) {
   GenericTaskBuilder builder_(_fbb);
   builder_.add_recurring(recurring);
   builder_.add_user(user);
@@ -134,6 +143,7 @@ inline flatbuffers::Offset<GenericTask> CreateGenericTask(
   builder_.add_time(time);
   builder_.add_file_info(file_info);
   builder_.add_id(id);
+  builder_.add_notify(notify);
   builder_.add_is_video(is_video);
   return builder_.Finish();
 }
@@ -148,7 +158,8 @@ inline flatbuffers::Offset<GenericTask> CreateGenericTaskDirect(
     int32_t mask = 0,
     const char *header = nullptr,
     const char *user = nullptr,
-    int32_t recurring = 0) {
+    int32_t recurring = 0,
+    bool notify = false) {
   auto file_info__ = file_info ? _fbb.CreateString(file_info) : 0;
   auto time__ = time ? _fbb.CreateString(time) : 0;
   auto description__ = description ? _fbb.CreateString(description) : 0;
@@ -164,7 +175,8 @@ inline flatbuffers::Offset<GenericTask> CreateGenericTaskDirect(
       mask,
       header__,
       user__,
-      recurring);
+      recurring,
+      notify);
 }
 
 inline const GenericData::GenericTask *GetGenericTask(const void *buf) {

src/argdialog.cpp

@@ -162,7 +162,11 @@ ArgDialog::ArgDialog(QWidget *parent) : QDialog(parent), ui(new Ui::ArgDialog),
     }
   });
 
-
+  QObject::connect(ui->notification, &QComboBox::currentTextChanged, this, [this](const QString& text) {
+    if (m_task->getType() == Scheduler::TaskType::GENERIC) {
+      m_task->setArgument("recurring", text.compare("Yes") == 0 ? true : false);
+    }
+  });
 }
 
 void ArgDialog::showEvent(QShowEvent* event) {

src/client.cpp

@@ -74,7 +74,10 @@ flatbuffers::Offset<GenericTask> createGenericTask(
       std::string{std::get<Scheduler::VariantIndex::QSTRING>(
         task->getTaskArgumentValue("user")).toUtf8().constData()}),
       std::get<Scheduler::VariantIndex::INTEGER>(
-        task->getTaskArgumentValue("recurring")));
+        task->getTaskArgumentValue("recurring")),
+      std::get<Scheduler::VariantIndex::BOOLEAN>(
+        task->getTaskArgumentValue("notify"))
+      );
 }
 
 flatbuffers::Offset<IGTask> createIGTask(

src/generic_task.cpp

@@ -6,13 +6,14 @@ using namespace Scheduler;
  * These values contained inside the TaskIndex namespace represent the order in which the tasks are to be stored.
  */
 namespace TaskIndex {
-static const uint8_t HEADER = 0;
-static const uint8_t DESCRIPTION = 1;
-static const uint8_t DATETIME = 2;
-static const uint8_t FILES = 3;
-static const uint8_t USER = 4;
-static const uint8_t IS_VIDEO = 5;
-static const uint8_t RECURRING = 6;
+static const uint8_t HEADER       = 0;
+static const uint8_t DESCRIPTION  = 1;
+static const uint8_t DATETIME     = 2;
+static const uint8_t FILES        = 3;
+static const uint8_t USER         = 4;
+static const uint8_t IS_VIDEO     = 5;
+static const uint8_t RECURRING    = 6;
+static const uint8_t NOTIFY       = 7;
 }  // namespace TaskIndex
 
 /**
@@ -35,6 +36,7 @@ void GenericTask::defineTaskArguments() {
   m_arguments.emplace_back(std::move(new TaskArgument{"user", Type::TEXT, QString{}}));
   m_arguments.emplace_back(std::move(new TaskArgument{"is_video", Type::BOOLEAN, bool{}}));
   m_arguments.emplace_back(std::move(new TaskArgument{"recurring", Type::INTEGER, findTaskFrequency("No")}));
+  m_arguments.emplace_back(std::move(new TaskArgument{"notify", Type::BOOLEAN, false}));
 }
 
 /**