fix: cache filing preference and add function outline

india_compliance/gst_india/api_classes/taxpayer_base.py

@@ -7,7 +7,7 @@
 
 import frappe
 import frappe.utils
-from frappe import _
+from frappe import _, request_cache
 from frappe.utils import add_to_date, cint, now_datetime
 
 from india_compliance.exceptions import (
@@ -465,6 +465,7 @@ def validate_auth_token(self):
 
         return
 
+    @request_cache
     def get_filing_preference(self, date=None):
         return self.get(
             action="GETPREF", params={"fy": self.get_fy(date)}, endpoint="returns"

india_compliance/gst_india/doctype/gst_return_log/gst_return_log.py

@@ -240,6 +240,9 @@ def _update_gstr_1_filed_upto(filing_date):
             getdate(f"{info['ret_prd'][2:]}-{info['ret_prd'][0:2]}-01")
         )
 
+        # is freq available in filing details ?
+        # if not, then fetch filing freq
+
         if key in gstr1_logs:
             if gstr1_logs[key] != info["arn"]:
                 frappe.db.set_value(DOCTYPE, key, filing_details)
@@ -260,6 +263,16 @@ def _update_gstr_1_filed_upto(filing_date):
         _update_gstr_1_filed_upto(filed_upto)
 
 
+def get_filing_frequency(gstin, period):
+    # is not Q1-M1 ? => fetch if available from Q1-M1
+    # if not, is it filed for the period ? => if not, then don't fetch and return nothing
+
+    # if filed, then fetch from the filed data
+
+    # cache the request for 2 minutes
+    pass
+
+
 def get_gst_return_log(posting_date, company_gstin):
     period = getdate(posting_date).strftime("%m%Y")
     if name := frappe.db.exists(DOCTYPE, f"GSTR1-{period}-{company_gstin}"):