cmoog / traderjoes

Price tracking program and website for Trader Joe's, updated daily.
https://traderjoesprices.com
MIT License
201 stars 15 forks source link

Omit extra html tags. #4

Closed ryanprince closed 7 months ago

ryanprince commented 7 months ago

Each table row in the output is wrapped in html tags.

    ...
    <html>
    <tr>
      <td><a href="https://traderjoes.com/home/products/pdp/020429" target="_blank">Spanish Manzanilla Olives with
          Pimento Paste</a></td>
      <td>2.99</td>
    </tr>

    </html>
    <html>
    <tr>
      <td><a href="https://traderjoes.com/home/products/pdp/020487" target="_blank">Panettone</a></td>
      <td>6.99</td>
    </tr>

    </html>
    ...

This change omits those extra tags. E.g.:

    ...
    <tr>
      <td><a href="https://traderjoes.com/home/products/pdp/020429" target="_blank">Spanish Manzanilla Olives with
          Pimento Paste</a></td>
      <td>2.99</td>
    </tr>
    <tr>
      <td><a href="https://traderjoes.com/home/products/pdp/020487" target="_blank">Panettone</a></td>
      <td>6.99</td>
    </tr>
    ...
cmoog commented 7 months ago

Oh whoops, this is embarrassing. Thank you for spotting this and implementing a fix!

ryanprince commented 7 months ago

No need for embarrassment. I was surprised to find out from this one that browsers didn't seem to mind.