fedora-hubs-widget-workshop

Clone

#3 Fix parameter name "text" -> "default"
Opened 6 years ago by pfrields. Modified 6 years ago
pfrields/fedora-hubs-widget-workshop fix-wrong-parameter-name  into  master

Merge
file modified
+2 -2 
@@ -38,8 +38,8 @@ 
 

              help="The Bugzilla email address.",
 

          )]
 

  

- You could have kept the ``text`` parameter and added ``email``, but since it's
 

- not going to be used anymore, just remove it.
 

+ You could have kept the ``default`` parameter and added ``email``, but
 

+ since it's not going to be used anymore, just remove it.
 

  

  

  Updating the view

no initial comment

Hmm this is actually correct, but not very clear I agree. In chapter 4, the parameters list only contained a dict with the name="text" key-value. In chapter 5, we replace that dict with one that has the name=email key-value. We could have left the other dict and it would not have been a problem, that's what I meant.
Am I clear? Can you think of a better way to express it?
Metadata
None
No Tags
Changes Summary 1
+2 -2
file changed
docs/realdata.rst
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.