instructure/canvas-lms · error · ImportError

No grade_publishing_status given for enrollment #

Error message

No grade_publishing_status given for enrollment #{enrollment_id}

What it means

Guard in SIS::GradePublishingResultsImporter::Work#add_grade_publishing_result: a grade publishing results row omits grade_publishing_status for the given enrollment_id, so ImportError is raised for that import row.

Solutions

  1. Ensure every CSV row has a grade_publishing_status value ('published' or 'error')
  2. Fix the status column mapping/order of arguments in the call
  3. Default or derive a status upstream when the source omits it, rather than passing blank

Example fix

// before
importer.add_grade_publishing_result(id, row['grade_publishing_status'])
// after
status = row['grade_publishing_status'].presence || 'error'
importer.add_grade_publishing_result(id, status)
Defensive patterns

Strategy: validation

Validate before calling

raise 'missing status' if grade_publishing_status.blank?
importer.add_grade_publishing_result(enrollment_id, grade_publishing_status)

Try / catch

begin
  importer.add_grade_publishing_result(id, status)
rescue SIS::Base::ImportError => e
  logger.warn("Skipping grade publishing result: #{e.message}")
end

Prevention

When it happens

Trigger: Calling `add_grade_publishing_result(enrollment_id, nil_or_empty, msg)`; grade publishing results CSV rows with an empty status column while enrollment_id is present.

Common situations: Export tooling omits the status for rows that never reached publishing; wrong header mapping so status parses as nil; script passes nil literal for the second argument.

Understand the failure class

Background: "must not be empty", "cannot be empty" — required-field validation errors across open-source libraries — this error's family across 41 libraries.

Related errors


AI-assisted analysis of instructure/canvas-lms@1c9f0bb801 (2026-09-15). Data as JSON: /api/errors/08327dbcee138936. Report an issue: GitHub.

Appendix: source

Thrown at lib/sis/grade_publishing_results_importer.rb:41

    def process
      importer = Work.new(@batch, @root_account, @logger)
      yield importer
      importer.success_count
    end

    class Work
      attr_accessor :success_count

      def initialize(batch, root_account, logger)
        @batch = batch
        @root_account = root_account
        @logger = logger
        @success_count = 0
      end

      def add_grade_publishing_result(enrollment_id, grade_publishing_status, message = nil)
        raise ImportError, "No enrollment_id given" if enrollment_id.blank?
        raise ImportError, "No grade_publishing_status given for enrollment #{enrollment_id}" if grade_publishing_status.blank?
        raise ImportError, "Improper grade_publishing_status \"#{grade_publishing_status}\" for enrollment #{enrollment_id}" unless %w[published error].include?(grade_publishing_status.downcase)

        if Enrollment.where(id: enrollment_id, root_account_id: @root_account)
                     .update_all(grade_publishing_status: grade_publishing_status.downcase,
                                 grade_publishing_message: message.to_s,
                                 updated_at: Time.now.utc) != 1
          raise ImportError, "Enrollment #{enrollment_id} doesn't exist"
        end

        @success_count += 1
      end
    end
  end
end

View on GitHub (pinned to 1c9f0bb801)