is it a good rails agile code?

I have two classes: User and Delivery.

class Ruser < ActiveRecord::Base
has_many :deliveries, :dependent => :destroy

class Delivery < ActiveRecord::Base
belongs_to :ruser

In the User show action I have put:

<%= render :partial => 'deliveries/form' %>

So that showing the user I can create a delivery for it.
Delivery form is:

<%= form_for(@delivery) do |f| %>
<% if @delivery.errors.any? %>
   <div id="error_explanation">
     <h2><%= pluralize(@delivery.errors.count, "error") %> prohibited
this delivery from being saved:</h2>

     <ul>
     <% @delivery.errors.full_messages.each do |msg| %>
       <li><%= msg %></li>
     <% end %>
     </ul>
   </div>
<% end %>

<div class="field">
   <%= f.label :delivered_at %>:
   <%= l Date.today %><br /><br />
</div>
<div class="field">
   <%= f.label :notes%><br />
   <%= f.text_area :notes %>
</div>

<%= f.hidden_field :ruser_id, :value => @ruser.id %>
<div class="actions">
   <%= f.submit %>
</div>
<% end %>

Is it a good code or there is a better mode to create a delivery for one user?