brynq-sdk-bob 2.8.2.dev0__py3-none-any.whl → 2.8.4__py3-none-any.whl

brynq_sdk_bob/people.py

@@ -52,30 +52,21 @@ class People:
         df.columns = df.columns.str.replace('/', '.', regex=False)
 
 
-        # Format custom columns in the same way as they were in the schema/request body.
-        # (e.g. 'identification.custom.field_1747318003716.value' -> 'identification.custom.field_1747318003716')
-        # But only if the target column doesn't already exist or is empty, to prioritize the cleaner name if it has value
+        # Clean up custom .value columns; keeps only one field per custom key. the .value column actually contains the value., other column is nested key.
         custom_cols = [col for col in df.columns if "custom" in str(col) and str(col).endswith('.value')]
         for col in custom_cols:
             new_col = col.removesuffix('.value')
-            # If the new column doesn't exist, rename it directly
+            # prefer non-empty clean column, otherwise upgrade .value data
             if new_col not in df.columns:
                 df = df.rename(columns={col: new_col})
-            # If it exists but is all empty/NA, and the .value column has data, overwrite it
             elif df[new_col].isna().all() and not df[col].isna().all():
                 df[new_col] = df[col]
                 df = df.drop(columns=[col])
-            # If both exist and have data, prefer the existing one (assuming cleaner format) but drop the .value duplicate if redundant
             else:
-                # If both exist, we might want to keep the existing one as is, and just drop the .value one to avoid confusion
-                # Or check if they are identical and drop one
+                # Drop identical/redundant .value col
                 if df[new_col].equals(df[col]):
                     df = df.drop(columns=[col])
-                else:
-                    # If they differ, we might want to keep the one with more data or default to existing behavior
-                    # For now, let's assume we trust the existing column if it has data.
-                    # Just drop the .value column if we aren't using it to overwrite
-                    pass
+                pass
 
 
         # A lot of fields from Bob are returned with only ID's. Those fields should be mapped to names. Therefore, we need to get the mapping from the named-lists endpoint.

brynq_sdk_bob/reports.py

@@ -25,13 +25,13 @@ class Reports:
 
         return df
 
-    def download(self, report_id: int | str = None) -> bytes:
+    def download(self, report_id: int | str = None, report_format: str = "csv") -> bytes:
         if report_id:
             url = f"{self.bob.base_url}company/reports/{report_id}/download"
         else:
             raise ValueError("Either report_id or report_name must be provided")
 
-        resp = self.bob.session.get(url=url, timeout=self.bob.timeout, params={"format": "csv"})
+        resp = self.bob.session.get(url=url, timeout=self.bob.timeout, params={"format": report_format})
         resp.raise_for_status()
         data = resp.content
 

brynq_sdk_bob/schemas/people.py

@@ -26,7 +26,7 @@ class PeopleSchema(BrynQPanderaDataFrameModel):
     first_name: Optional[Series[String]] = pa.Field(coerce=True, nullable=True, description="First Name", alias="firstName", metadata={"api_field": "root.firstName"})
     full_name: Optional[Series[String]] = pa.Field(coerce=True, nullable=True, description="Full Name", alias="fullName", metadata={"api_field": "root.fullName"})
     # the date is in DD/MM/YYYY format,
-    personal_birth_date: Optional[Series[DateTime]] = pa.Field(coerce=True, nullable=False, description="Personal Birth Date", alias="personal.birthDate")
+    personal_birth_date: Optional[Series[DateTime]] = pa.Field(coerce=True, nullable=True, description="Personal Birth Date", alias="personal.birthDate")
     personal_pronouns: Optional[Series[String]] = pa.Field(coerce=True, nullable=True, description="Personal Pronouns", alias="personal.pronouns")
     personal_honorific: Optional[Series[String]] = pa.Field(coerce=True, nullable=True, description="Personal Honorific", alias="personal.honorific")
     personal_nationality: Optional[Series[object]] = pa.Field(coerce=True, check_name=check_list, description="Personal Nationality", alias="personal.nationality")

{brynq_sdk_bob-2.8.2.dev0.dist-info → brynq_sdk_bob-2.8.4.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: brynq_sdk_bob
-Version: 2.8.2.dev0
+Version: 2.8.4
 Summary: Bob wrapper from BrynQ
 Author: BrynQ
 Author-email: support@brynq.com

{brynq_sdk_bob-2.8.2.dev0.dist-info → brynq_sdk_bob-2.8.4.dist-info}/RECORD

@@ -7,8 +7,8 @@ brynq_sdk_bob/employment.py,sha256=uNllQrIBbo8yPG_2-ln1PWeWUFU672T289PpWWvL-V8,7
 brynq_sdk_bob/named_lists.py,sha256=ksLXV2ysBFegq4gZiiaC56gjkgdnPzL7WajZTGvjYIM,1069
 brynq_sdk_bob/payments.py,sha256=HVXoMyA0Jyhyk5EcgCU2OQfdbMtElhdLfdBKgI9xC4Q,3922
 brynq_sdk_bob/payroll_history.py,sha256=wHo6da7kLDe1ViL4egyMdyJBMZnWVhwjNjmh4cTCTeY,3972
-brynq_sdk_bob/people.py,sha256=2vgut_HnlwzWI6-yzdeQrPRQSz5YBNsD0_QqVtwrXPs,6810
-brynq_sdk_bob/reports.py,sha256=fiYzd6037siGl6c4KQLE7F4CeFRRDVDgSIVcWVVWsvI,1209
+brynq_sdk_bob/people.py,sha256=t1A1dABX6UZ0pyLTGOL-Sp5pHY630KWIyIO3JQ_Pjdk,5970
+brynq_sdk_bob/reports.py,sha256=Tawmqm_ZmQ487loyk-29-A_fTCrgImbWCEf6zfwuaq4,1245
 brynq_sdk_bob/salaries.py,sha256=BGQm-PT9QuKKJ9DP5nX6wmC8SZRAlm9M9I2EJhoZaII,1523
 brynq_sdk_bob/timeoff.py,sha256=NbBZ39qy9D7jbS_z9bpmB-BKNuUGmNrkYTbEw034tZ0,3339
 brynq_sdk_bob/work.py,sha256=0bVZkQ0I6z-z2_ql-EsOpFExx8VgsJvpcCQdOfiJYQM,712
@@ -19,11 +19,11 @@ brynq_sdk_bob/schemas/employment.py,sha256=KUGbT8hjYuQIN-3BGlIrBBswXT6fSPkLtn5bl
 brynq_sdk_bob/schemas/named_lists.py,sha256=HJBRKrAI2vhrkq-5MVXqQcmpGNzFtoOnaZI2Ii_6_vs,725
 brynq_sdk_bob/schemas/payments.py,sha256=LrSr8WApYxqbMDmhYmh0EISEbWJinWovsULZV6sItG0,4290
 brynq_sdk_bob/schemas/payroll_history.py,sha256=JdAq0XaArHHEw8EsXo3GD0EhSAyBhPtYQMmdvjCiY8g,806
-brynq_sdk_bob/schemas/people.py,sha256=Zs58mO4i8M-u-DLWRS0IHbyDNiwrhxSxOX8t3mLr2fw,40150
+brynq_sdk_bob/schemas/people.py,sha256=42BJVgJmT-h5kzuQl6iI7wZDSGNA0KTQQVIAqeeyHNk,40149
 brynq_sdk_bob/schemas/salary.py,sha256=TSaM1g92y3oiDcUrfJW7ushgKZenI9xB6XW3kKuU0dE,4540
 brynq_sdk_bob/schemas/timeoff.py,sha256=uswH42djiiWE2H0wXKHoneDvOJQ5BlaIeNdNl-kYd-s,4117
 brynq_sdk_bob/schemas/work.py,sha256=YgtBJ0WXJOq55bFlT_kY_IbHh0SlQEtaa0W8vms-xA4,3048
-brynq_sdk_bob-2.8.2.dev0.dist-info/METADATA,sha256=XOtNxJvNDOa2913ueNfR132HeYHuBvEk92K4GhZJr_I,376
-brynq_sdk_bob-2.8.2.dev0.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-brynq_sdk_bob-2.8.2.dev0.dist-info/top_level.txt,sha256=oGiWqOuAAiVoLIzGe6F-Lo4IJBYz5ftOwBft7HtPuoY,14
-brynq_sdk_bob-2.8.2.dev0.dist-info/RECORD,,
+brynq_sdk_bob-2.8.4.dist-info/METADATA,sha256=i8xKzW4Se01HsfeIOHSe_uHIxGVpornQ0Elqhdz0kHE,371
+brynq_sdk_bob-2.8.4.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+brynq_sdk_bob-2.8.4.dist-info/top_level.txt,sha256=oGiWqOuAAiVoLIzGe6F-Lo4IJBYz5ftOwBft7HtPuoY,14
+brynq_sdk_bob-2.8.4.dist-info/RECORD,,