Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Documentation: "asynchronously" is misleading here #6753

Merged
merged 1 commit into from

2 participants

@schnittchen

make it clearer that subscribers are not notified asynchronously, but saved for later use.

Schnittchen Documentation: make it clearer that subscribers are not notified asyn…
…chronously, but saved for later use.
c779a47
@josevalim
Owner

Yes, it was async in the first version, but no longer.

@josevalim josevalim merged commit 27846d8 into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jun 16, 2012
  1. Documentation: make it clearer that subscribers are not notified asyn…

    Schnittchen authored
    …chronously, but saved for later use.
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  activesupport/lib/active_support/notifications.rb
View
2  activesupport/lib/active_support/notifications.rb
@@ -33,7 +33,7 @@ module ActiveSupport
# end
#
# That code returns right away, you are just subscribing to "render" events.
- # The block will be called asynchronously whenever someone instruments "render":
+ # The block is saved and will be called whenever someone instruments "render":
#
# ActiveSupport::Notifications.instrument("render", :extra => :information) do
# render :text => "Foo"
Something went wrong with that request. Please try again.