You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
mailroom with dictionaries: remove reference to "letter" #46
confusing to students mention of letter (which used to be combined with files/letter feature)
Try to use a dict and the .format() method to produce the letter as one big template, rather than building up a big string that produces the letter in parts.
I think learning how to use ** with .format is still a good idea.
And I don't understand the confusion -- this isn't changing / adding an features, simply suggesting a way to use template, .format() and a dict to do the same thing.
But if you can think of a way to make it more clear, sure.
the term "letter" isn't something that's a feature until later so the confusion is the terminology used -- "produce a letter" but we don't ask to produce a letter feature until later.
"send letters to all donors" menu option is added later on now.
Activity
PythonCHB commentedon Dec 27, 2020
I think learning how to use ** with .format is still a good idea.
And I don't understand the confusion -- this isn't changing / adding an features, simply suggesting a way to use template, .format() and a dict to do the same thing.
But if you can think of a way to make it more clear, sure.
natasha-aleksandrova commentedon Dec 31, 2020
the term "letter" isn't something that's a feature until later so the confusion is the terminology used -- "produce a letter" but we don't ask to produce a letter feature until later.
"send letters to all donors" menu option is added later on now.
PythonCHB commentedon Dec 31, 2020
ahh -- good catch, I lost track of what's when. I'll try to update that soon.
Odd that I didn't have any students complain about this in my class ....