RocHack / bb

Command line Blackboard client
MIT License
72 stars 8 forks source link

Announcements #25

Closed joeljk13 closed 9 years ago

joeljk13 commented 9 years ago

It's pretty hackish and not very pretty, but even as it is now it can be helpful.

2 things it definitely could use but doesn't have:

clehner commented 9 years ago

Looks great! Is there anything to be done about HTML entity decoding? I currently see several items like:

Tuesday 1/20: Fundamentals of Tai Chi & Qigong 
joeljk13 commented 9 years ago

Yeah that comes from the hackish implementation - all I did was remove tags like <tag attr="some text"> and </tag>, change <br /> to a new line, and remove initial spaces on each line. Of course we could do something like sed 's/&amp;/\&/g' for each case, but that'll get very messy.

joeljk13 commented 9 years ago

I just realized that I had 2 commits updating whitespace, so I've squashed them together.

joeljk13 commented 9 years ago

I'm calling this good enough to merge; it works and is helpful despite its lacks.

Piping to less works well to deal with the lack of limiting.

clehner commented 9 years ago

:+1: