#3609 - Déplacement de l'URL qui envoie vers la procédure

[Keirua]

#3609:

• plus de lien à la création de la démarche
• le lien est demandé lorsqu'on publie la démarche
• on peut modifier ce lien dans l'onglet "description" si la démarche est publiée

[kemenaran]

Il y a une raison particulière de mettre locked? plutôt que published || archived ?

Ah, mais sinon on pourrait faire plus simple : mettre juste if !@procedure.new?. Comme ça la première fois qu'on remplit le formulaire, c'est masqué – mais ensuite on a le champ. Ça éviterait qu'un champ apparaisse et disparaisse mystérieusement en fonction des états.

[Keirua]

non, j'ai utilisé locked? car j'ai vu que c'est ce qu'il y a au dessus pour un cas d'usage du même ordre. mais ça appelle publiee_ou_archivee? en interne.

L'utilisation de procedure.new semble moins dépendante de l'état en effet, je vais l'ajouter.

[Keirua]

j'ai finalement opté pour procedure.created_at.present? car new? pas existant sur cette procédure. C'est surprenant, car on a pourtant forcément une procédure via ActiveRecord:

# Admin::ProceduresController#new
  def new
    @procedure ||= Procedure.new(for_individual: true)
    @availability = Procedure::PATH_AVAILABLE
  end

[kemenaran]

@Keirua friendly ping :) Cette PR est toujours d'actualité ?

[Keirua]

Oui, c'est juste que je n'avais pas pris le temps de rendre passant les tests autos qui étaient cassés suite aux changements de can_publish?