dscf-banking 0.2.8 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bd6bfb0114ad393bd22ae2bf1db82507a7e50299aa6f7ac0c433e53ae3a24e91
-  data.tar.gz: 97773b5c82e95ab7d227d9473ce82658ce4df88966506d0354ee10d360749023
+  metadata.gz: 77261771378e8c5903e5f77c66f3919e7528514bd7c8a44070dbe07c61c5aced
+  data.tar.gz: 4db5b73418dc1a2625e1d01ba578d39e38655acf41361320735f75792eb986a7
 SHA512:
-  metadata.gz: 41ff719fa8dceb6e423d57c80bc8d9b96aeb2ab619ab94eccffb24bc1cbe4ce852b160ecde131084e11902aedf95df074af6b3589ffe3335f28cfcf10abccdcc
-  data.tar.gz: 5db3b15472498d3509cfe556ade8bf5e64ff7261cd3e0a338f898351c8a59110fddba5d4a04f1857a106508a5154c88d8dcd3964fbcaa5b22800478ede721e49
+  metadata.gz: e23b8ca0fe1ed1ba9320edb49c4d22685547ea1a66fe5bcb7944ffafcb29ba9525107e42efe34f570006dcfe5364e5f70fb4d4be20eb4c9c51c3057f72883eff
+  data.tar.gz: 475ee3557215bb66076ad7b5bd414a23c3e41422200bc9c64ebb1b21032c9f737c9407b31f4bbf8745ab562de8444d07d22c01d7c42adab7825830c35140365f

data/app/controllers/dscf/banking/accounts_controller.rb

@@ -195,6 +195,24 @@ module Dscf::Banking
       }, status: :not_found
     end
 
+    def me
+      accounts = Dscf::Banking::Account
+        .joins(:application)
+        .where("dscf_banking_applications.user_id = ?", current_user.id)
+        .includes(:application, :virtual_account_product)
+
+      # Apply ordering
+      order_column = allowed_order_columns.include?(params[:order_by]) ? params[:order_by] : "created_at"
+      order_direction = %w[asc desc].include?(params[:order_direction]) ? params[:order_direction] : "desc"
+      accounts = accounts.order("dscf_banking_accounts.#{order_column} #{order_direction}")
+
+      render json: {
+        success: true,
+        data: accounts.map { |account| account_data(account) },
+        message: "User accounts retrieved successfully"
+      }
+    end
+
     private
 
     def account_data(account)

data/app/controllers/dscf/banking/applications_controller.rb

@@ -22,30 +22,6 @@ module Dscf::Banking
         resubmit: { status: "submitted", update_model: true }
       }
 
-    # Override perform_review_action to sync application status with review status
-    def perform_review_action(action_name, action_config, context_name)
-      result = super
-      
-      # After successful review action, update the application's status to match
-      if result.is_a?(Hash) && result[:success] && @obj
-        review_status = @obj.review_status
-        application_status_map = {
-          "submitted" => "submitted",
-          "under_review" => "under_review",
-          "approved" => "approved",
-          "rejected" => "rejected",
-          "request_modification" => "request_modification"
-        }
-        
-        new_status = application_status_map[review_status]
-        if new_status && @obj.status != new_status
-          @obj.update_column(:status, Application.statuses[new_status])
-        end
-      end
-      
-      result
-    end
-
     private
 
   def application_data(application)

data/app/controllers/dscf/banking/documents_controller.rb

@@ -4,6 +4,42 @@ module Dscf::Banking
 
     before_action :find_application
 
+    def index
+      super do
+        documents = @application.documents
+        options = { each_serializer: DocumentSerializer }
+        [documents, options]
+      end
+    end
+
+    def show
+      super do
+        document = @application.documents.find(params[:id])
+        options = { serializer: DocumentSerializer }
+        [document, options]
+      end
+    end
+
+    def create
+      document = @application.documents.build(model_params)
+      
+      if document.save
+        # Attach file if provided
+        document.files.attach(params[:document][:file]) if params[:document][:file].present?
+        
+        render json: {
+          success: true,
+          data: DocumentSerializer.new(document).as_json
+        }, status: :created
+      else
+        render json: {
+          success: false,
+          error: "Failed to create document",
+          errors: document.errors.full_messages
+        }, status: :unprocessable_entity
+      end
+    end
+
     private
 
     def find_application
@@ -49,5 +85,9 @@ module Dscf::Banking
         update: []
       }
     end
+
+    def serializer_class
+      Dscf::Banking::DocumentSerializer
+    end
   end
 end

data/app/models/dscf/banking/application.rb

@@ -22,6 +22,7 @@ module Dscf::Banking
     before_validation :generate_application_number, if: -> { new_record? && application_number.blank? }
     before_update :set_timestamps_on_status_change
     after_commit :create_account_if_approved, on: :update
+    after_find :sync_status_if_needed
 
     def approve!(approved_by = nil)
       update!(status: :approved, completed_at: Time.current)
@@ -45,7 +46,12 @@ module Dscf::Banking
 
     def review_status
       latest_review = reviews.where(context: "default").order(created_at: :desc).first
-      latest_review&.status || status
+      review_st = latest_review&.status || status
+
+      # Sync application status with review status
+      sync_status_with_review(review_st) if latest_review
+
+      review_st
     end
 
     def self.ransackable_attributes(auth_object = nil)
@@ -77,10 +83,57 @@ module Dscf::Banking
     end
 
     def create_account_if_approved
-      return unless review_status == "approved"
+      # First sync the status with the latest review
+      current_review_status = review_status  # This will call sync_status_with_review
+
+      reload  # Reload to get the updated status
+
+      return unless status_approved?
       return if has_account?
 
       AccountCreationService.call(self)
     end
+
+    def sync_status_with_review(review_st)
+      return if @syncing_status
+
+      status_map = {
+        "submitted" => "submitted",
+        "under_review" => "under_review",
+        "approved" => "approved",
+        "rejected" => "rejected",
+        "request_modification" => "request_modification"
+      }
+
+      mapped_status = status_map[review_st]
+      if mapped_status && status != mapped_status && !new_record?
+        @syncing_status = true
+
+        # Prepare attributes for update
+        attributes_to_update = { status: mapped_status }
+
+        # If rejecting, also set rejection_reason from review feedback
+        if mapped_status == "rejected"
+          latest_review = reviews.where(context: "default", status: "rejected").order(created_at: :desc).first
+          if latest_review&.feedback.present?
+            attributes_to_update[:rejection_reason] = latest_review.feedback
+          end
+        end
+
+        update!(attributes_to_update)
+        @syncing_status = false
+      end
+    end
+
+    def sync_status_if_needed
+      return if @status_synced || new_record?
+
+      latest_review = reviews.where(context: "default").order(created_at: :desc).first
+      if latest_review && latest_review.status != status
+        sync_status_with_review(latest_review.status)
+      end
+
+      @status_synced = true
+    end
   end
 end

data/app/serializers/dscf/banking/application_serializer.rb

@@ -9,7 +9,7 @@ module Dscf::Banking
     belongs_to :assigned_branch_manager, optional: true
     has_one :account, serializer: AccountSerializer, if: :has_account?
     has_many :reviews, serializer: Dscf::Core::ReviewSerializer
-    has_many :documents, serializer: DocumentSerializer
+    has_many :documents, serializer: Dscf::Banking::DocumentSerializer
 
     def has_account?
       object.has_account?

data/app/serializers/dscf/banking/document_serializer.rb

@@ -1,8 +1,15 @@
 module Dscf::Banking
   class DocumentSerializer < ActiveModel::Serializer
-    attributes :id, :document_type, :is_verified, :verified_at, :metadata, :created_at, :updated_at
+    attributes :id, :document_type, :is_verified, :verified_at, :metadata, :file_urls, :created_at, :updated_at
 
-    # Remove polymorphic association that causes ActiveStorage issues
-    # belongs_to :verified_by, polymorphic: true, optional: true
+    def file_urls
+      return [] unless object.files.attached?
+      
+      object.files.map do |file|
+        Rails.application.routes.url_helpers.rails_blob_path(file, only_path: true)
+      end
+    rescue StandardError
+      []
+    end
   end
 end

data/app/services/dscf/banking/account_creation_service.rb

@@ -47,7 +47,7 @@ module Dscf::Banking
     end
 
     def generate_account_name
-      "Test Account - #{@application.application_number}"
+      "#{@application.virtual_account_product.product_name}"
     end
   end
 end

data/config/initializers/review_callbacks.rb

@@ -0,0 +1,23 @@
+Rails.application.config.to_prepare do
+  ActiveSupport::Notifications.subscribe("review.created") do |name, start, finish, id, payload|
+    reviewable = payload[:reviewable]
+    
+    if reviewable.is_a?(Dscf::Banking::Application)
+      reviewable.reload
+      review_status = reviewable.review_status
+      
+      status_map = {
+        "submitted" => "submitted",
+        "under_review" => "under_review",
+        "approved" => "approved",
+        "rejected" => "rejected",
+        "request_modification" => "request_modification"
+      }
+      
+      mapped_status = status_map[review_status]
+      if mapped_status && reviewable.status != mapped_status
+        reviewable.update!(status: mapped_status)
+      end
+    end
+  end
+end

data/config/initializers/review_extensions.rb

@@ -0,0 +1,36 @@
+module Dscf::Banking
+  module ReviewExtensions
+    extend ActiveSupport::Concern
+
+    included do
+      after_create :sync_application_status, if: -> { reviewable.is_a?(Dscf::Banking::Application) }
+    end
+
+    private
+
+    def sync_application_status
+      return unless reviewable.is_a?(Dscf::Banking::Application)
+
+      application = reviewable
+      review_status = application.review_status
+
+      status_map = {
+        "submitted" => "submitted",
+        "under_review" => "under_review",
+        "approved" => "approved",
+        "rejected" => "rejected",
+        "request_modification" => "request_modification"
+      }
+
+      mapped_status = status_map[review_status]
+      if mapped_status && application.status != mapped_status
+        application.update!(status: mapped_status)
+      end
+    end
+  end
+end
+
+# Extend the Dscf::Core::Review model with our extensions
+Rails.application.config.to_prepare do
+  Dscf::Core::Review.include(Dscf::Banking::ReviewExtensions) if defined?(Dscf::Core::Review)
+end

data/config/routes.rb

@@ -1,5 +1,8 @@
 Dscf::Banking::Engine.routes.draw do
   resources :accounts do
+    collection do
+      get :me
+    end
     member do
       post :activate
       post :suspend

data/lib/dscf/banking/version.rb

@@ -1,5 +1,5 @@
 module Dscf
   module Banking
-    VERSION = "0.2.8"
+    VERSION = "0.3.0"
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: dscf-banking
 version: !ruby/object:Gem::Version
-  version: 0.2.8
+  version: 0.3.0
 platform: ruby
 authors:
 - Eyosiyas Mekbib
 bindir: bin
 cert_chain: []
-date: 2025-10-15 00:00:00.000000000 Z
+date: 2025-10-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -482,6 +482,8 @@ files:
 - app/services/dscf/banking/deposit_service.rb
 - app/services/dscf/banking/transfer_service.rb
 - app/services/dscf/banking/withdrawal_service.rb
+- config/initializers/review_callbacks.rb
+- config/initializers/review_extensions.rb
 - config/locales/en.yml
 - config/routes.rb
 - db/migrate/20250830211002_create_dscf_banking_product_categories.rb