search

jquast / x84

A python telnet/ssh server for modern terminals. In spirit of classic software such as ami/x, teleguard, renegade, iniquity.
http://x84.readthedocs.org/
Other
373 stars 56 forks source link

Tetris and Weather Fixups #265

Closed M-griffin closed 9 years ago

M-griffin commented 9 years ago

Summary tetris.py echo(syncterm_setfont('cp437')) to echo_unbuffered(syncterm_setfont('cp437')) Original buffered settings makes this command output after the ANSI screen is drawn. It should push out prior to displaying the ANSI screen.

Summary weather.py

  1. loop was not saving new searches or prompting the user to save because it was outside of the loop. This affected current users since it was not updating and only retrieving their last save. And affected new users because it would never save a new location.
  2. Once this was fixed I noticed Accu Weather API doesn't return postal code anymore on findcity. This is causing the get.location to retrieve a bad value on postal lookup for pre-populating the search field. Changing this to City, State lookup solves the issues and looks nice too.
M-griffin commented 9 years ago

New pull request with change log. :) Hope this is correct now.

landscape-bot commented 9 years ago

Code Health Code quality remained the same when pulling a8936cb on M-griffin:master into e8d4cdf on jquast:master.