Fix content description when less then 100 chars in note

[RobbertWolfs]

When the note content is below 100 chars we currently show 'No content' instead of the content itself

[RobbertWolfs]

Note with 100+ chars -> we slice the content with ... dots

Note with less than 100 chars -> we show "No content" even when we have content

After the change

[kentcdodds]

Thanks for this! I'll want to have this change happen for all following exercises so we don't end up with a diff that incorrectly "reverts" this change in the next exercise here :)

[RobbertWolfs]

I'll update it in the next exercises as well

[RobbertWolfs]

@kentcdodds Fixed it on the next exercises as well.

Is it useful to create a util function for this or is this overkill for the exercise?

function generateNoteSummary({
  content,
  maxLength = 100,
  sliceLength = 97
}: NoteSummaryParams): string {
  if (!content) {
    return 'No content';
  }

  if (content.length > maxLength) {
    return content.slice(0, sliceLength) + '...';
  }

  return content;
}

[RobbertWolfs]

@kentcdodds I also noticed that in the following workshops we also have the same "code" so is it necessary to also fix it there or ?

https://github.com/epicweb-dev/web-forms/blob/8431630659fe45a4236b2e4ad78a35d02a8049c2/exercises/01.form-validation/01.problem.form-validation/app/routes/users%2B/%24username_%2B/notes.%24noteId.tsx#L82