Skip to content
Snippets Groups Projects

Set appsignal transaction type

Closed Set appsignal transaction type
tr-set-appsignal-transaction-type into master
Closed automation_daan requested to merge tr-set-appsignal-transaction-type into master

Created by: timraasveld

Merge request reports

Checking pipeline status.

Closed by automation_daanautomation_daan 7 years ago (Oct 23, 2017 2:20pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
lib/roqua/support/errors.rb
Unable to load the diff
  • automation_daan
    automation_daan @automation ·
    Author Owner

    Created by: hampei

    Roqua zit op 2.0.4, die versie wil hier 3 argumenten hebben. Stiekem gebruiken we deze code nergens binnen roqua denk ik, omdat we altijd of binnen een een webrequest zitten of binnen een delayed_job, waarbij we in de if-tak zitten.

    Denk wel dat als we in deze tak komen dat we er vanuit kunnen gaan dat we in een background-task zitten, dus hoef je niet perse mee te geven.

    Appsignal::Transaction.create(SecureRandom.uuid, Appsignal::Transaction::BACKGROUND_JOB, Appsignal::Transaction::GenericRequest.new({}))

    Al hebben we dus een andere oplossing nodig voor het feit dat we maar 1 melding binnen krijgen voor de hele prepare_subscriptions job. Mogelijk door een vlag new_transaction mee te geven ofzo (of iets minder appsignal-specifieks met die strekking).

  • automation_daan closed

    closed

    • Please register or sign in to reply