gideonshaked/nextinspace

Never miss a launch. 🚀

nextinspace's Issues

Wrong error raised when API unavailable; should be API error

When there is a connection issue, the stack trace has nothing to do with a connection issue.

Catch the ratelimit exception

Traceback (most recent call last):
  File "/Users/derkweijers/.pyenv/versions/3.9.0/lib/python3.9/site-packages/nextinspace/api.py", line 230, in get_data
    response.raise_for_status()
  File "/Users/derkweijers/.pyenv/versions/3.9.0/lib/python3.9/site-packages/requests/models.py", line 941, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 429 Client Error: Too Many Requests for url: https://ll.thespacedevs.com/2.0.0/event/upcoming/?limit=1

I think it would be nicer to catch the ratelimit exception and return a cleaner error :)

Conversion into local time is not done correctly

The time in the output is supposedly local (in my case ET), but is actually the time of the item in UTC.
Example:

┌────────────────────────────────────────────────────────────────────────────────────────┐
│Atlas V 531 | NROL-101                                                                  │
│Space Launch Complex 41, Cape Canaveral, FL, USA                                        │
│                                                                                        │
│    Tue November 03, 2020 10:58 PM EST                                                  │
│    Launch Type: Government/Top Secret                                                  │
│                                                                                        │
│    Classified payload for the U.S. National Reconnaissance Office.                     │
└────────────────────────────────────────────────────────────────────────────────────────┘

This SHOULD be:

┌────────────────────────────────────────────────────────────────────────────────────────┐
│Atlas V 531 | NROL-101                                                                  │
│Space Launch Complex 41, Cape Canaveral, FL, USA                                        │
│                                                                                        │
│    Tue November 03, 2020 5:58 PM EST                                                   │
│    Launch Type: Government/Top Secret                                                  │
│                                                                                        │
│    Classified payload for the U.S. National Reconnaissance Office.                     │
└────────────────────────────────────────────────────────────────────────────────────────┘

To confirm see the launch page on the LL2 API.

URL is not correct

nextinspace: 404 Client Error: Not Found for url: https://ll.thespacedevs.com/2.0.1/event/upcoming?limit=1

It looks like we need https://ll.thespacedevs.com/2.1.0/event/upcoming/?limit=1

Feature Request: I've made an Arch Linux PKGBUILD for this, can I put it on the AUR?

I just wanted your permission if I could do this :)

Feature request: Multi-column display

4 to 5 columns could easily fit on a 32" monitor @ 1440p resolution.
Any chance of getting to to print more than 1 column?

Feature Request: Plain text output option

The panes are neat, but it would be nice to have a plain text output mode that just outputs text, no panes, no colors.

Dealing with rate limits

I started receiving HTTP 429 errors after just a couple of invocations of the program. Would it be possible to cache the data somewhere for a limited time so it doesn't need to be re-requested when just trying out different combinations of parameters?

Text doesn't wrap inside the container

Not a big deal, but as you can see the text runs outside the container.