search

gtalarico / aec-startups

List of AEC Startups
https://www.aecstartups.com
MIT License
58 stars 16 forks source link

Can't see any entries after "n" #66

Closed theo-armour closed 3 years ago

theo-armour commented 3 years ago

List is being truncated after "nPlan"

image

gtalarico commented 3 years ago

You are right - I will look into this tomorrow.

gtalarico commented 3 years ago

lambda was capping records to first 100. fixed here: https://github.com/gtalarico/aec-startups/commit/598c448c631a84f82cfa9c84e421456dd122c5ab

invalidated cache to fix should be fixed

nplan still shows last, not sure why, but at least all records seem to be coming through, including the W's.

image

theo-armour commented 3 years ago

@gtalarico

nPlan is probably last maybe because Linux sorts lower case letters after uppercase letters

Nonetheless, thanks for the fix!

gtalarico commented 3 years ago

ha! you are right about case but not linux -> JS https://github.com/gtalarico/aec-startups/blob/ef368c1872fafa199ba3b92c9a726e5d9b82a612/src/App.vue#L71

theo-armour commented 3 years ago

@gtalarico

My doh!

What happens if you add .toLowerCase() to each of the titles?

gtalarico commented 3 years ago

it would fix it