#1642 & #1559 (#1647)

* #1641, #1642 & #1559
 - #1641
    - removed character limitation for work_statuses.description & work_issues.description
 - #1642
    - Updated existing staff entries email to lower case
    - Refactored code to set email to lower case when adding/updating staffs
 - #1559
    - Fixed the issue where task template upload was failing when `tips` column was empty

* lint fixes

* moved lower case conversion to schema for staff validation and existance check APIs

* remove #1641 related changes

* revert workplan issues form & status form to include character limits

* re-added validation rule to WorkStatusParameterSchema

* merge migrations

* resolved conflicts, added `email` in data check

* lint fix

Author: salabh-aot

epictrack-api/migrations/versions/097d41841ba4_convert_staff_emails_to_lower_case.py

@@ -0,0 +1,43 @@
+"""Convert staff emails to lower case
+
+Revision ID: 097d41841ba4
+Revises: 65cb91595d6a
+Create Date: 2024-01-11 14:04:10.156478
+
+"""
+import sqlalchemy as sa
+from alembic import op
+from sqlalchemy.sql import column, table
+
+
+# revision identifiers, used by Alembic.
+revision = '097d41841ba4'
+down_revision = '65cb91595d6a'
+branch_labels = None
+depends_on = None
+
+
+def upgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    conn = op.get_bind()
+    staff_table = table('staffs',
+        column('id', sa.Integer),
+		column('name', sa.String),
+		column('phone',sa.String),
+		column('email', sa.String),
+		column('is_active', sa.Boolean),
+		column('position_id', sa.Integer)
+    )
+
+    conn.execute(
+            staff_table.update()\
+                .values(email=sa.func.lower(staff_table.c.email))
+        )
+
+    # ### end Alembic commands ###
+
+
+def downgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    pass
+    # ### end Alembic commands ###

epictrack-api/migrations/versions/52b37bc04925_merging_097d41841ba4_and_09a79ca8e8fd.py

@@ -0,0 +1,24 @@
+"""merging 097d41841ba4 and 09a79ca8e8fd
+
+Revision ID: 52b37bc04925
+Revises: 097d41841ba4, 09a79ca8e8fd
+Create Date: 2024-01-15 10:02:26.499111
+
+"""
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision = '52b37bc04925'
+down_revision = ('097d41841ba4', '09a79ca8e8fd')
+branch_labels = None
+depends_on = None
+
+
+def upgrade():
+    pass
+
+
+def downgrade():
+    pass

epictrack-api/src/api/models/work_status.py

@@ -14,7 +14,7 @@
 """Model to handle all operations related to WorkStatus."""
 from __future__ import annotations
 
-from typing import List, Dict
+from typing import Dict, List
 
 from sqlalchemy import Boolean, Column, DateTime, ForeignKey, Integer, String, desc
 from sqlalchemy.orm import relationship

epictrack-api/src/api/schemas/request/staff_request.py

@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 """Staff resource's input validations"""
-from marshmallow import fields, validate
+from marshmallow import fields, pre_load, validate
 
 from api.schemas.validators import Phone
 
@@ -47,6 +47,14 @@ class StaffExistanceQueryParamSchema(RequestQueryParameterSchema):
         missing=None,
     )
 
+    @pre_load
+    def convert_email_to_lower(self, data, **kwargs):  # pylint: disable=unused-argument
+        """Converts staff email into lower case string"""
+        data = dict(data)
+        if "email" in data:
+            data["email"] = data["email"].lower()
+        return data
+
 
 class StaffByPositionsQueryParamSchema(BasicRequestQueryParameterSchema):
     """Staff by positions query parameter"""
@@ -88,10 +96,16 @@ class StaffBodyParameterSchema(RequestBodyParameterSchema):
     )
 
     is_active = fields.Boolean(
-        metadata={"description": "Active status of the staff"},
-        required=True
+        metadata={"description": "Active status of the staff"}, required=True
     )
 
+    @pre_load
+    def convert_email_to_lower(self, data, **kwargs):  # pylint: disable=unused-argument
+        """Converts staff email into lower case string"""
+        if "email" in data:
+            data["email"] = data["email"].lower()
+        return data
+
 
 class StaffEmailPathParameterSchema(RequestPathParameterSchema):
     """Staff email path parameter schema"""
@@ -101,3 +115,11 @@ class StaffEmailPathParameterSchema(RequestPathParameterSchema):
         validate=validate.Email(),
         required=True,
     )
+
+    @pre_load
+    def convert_email_to_lower(self, data, **kwargs):  # pylint: disable=unused-argument
+        """Converts staff email into lower case string"""
+        data = dict(data)
+        if "email" in data:
+            data["email"] = data["email"].lower()
+        return data

epictrack-api/src/api/schemas/request/task_request.py

@@ -87,8 +87,7 @@ class TaskBodyParameterSchema(RequestBodyParameterSchema):
 
     tips = fields.Str(
         metadata={"description": "Practical info on why/how to do the task"},
-        # validate=validate.Length(max=150),
-        required=True,
+        allow_none=True
     )
 
     number_of_days = fields.Int(