Skip to content

Reworked: changed: geolocation to be local hosted to skip rate limits#48

Merged
EvanStreams merged 3 commits into
hackclub:mainfrom
EliasL-git:main
May 11, 2026
Lines changed: 756 additions & 189 deletionsions
Merged

Reworked: changed: geolocation to be local hosted to skip rate limits#48
EvanStreams merged 3 commits into
hackclub:mainfrom
EliasL-git:main

Conversation

@EliasL-git
Copy link
Copy Markdown
Contributor

@EliasL-git EliasL-git commented May 8, 2026

Reworked:

Geolocation:* Switched to a local, self-hosted geolocation service using geoip-lite to eliminate
third-party API dependencies, CORS blocking, and rate-limit issues..
Visual Overhaul:**
Premium UI: Re-imagined the Changelog as a modern, brand-consistent Timeline*..
Hack Club Theming:* Replaced the previous purple gradients with Hack Club’s signaturee
Red-to-Orange brand palette throughout the onboarding and UI to ensure a cohesive, on-brand look..
Accessibility:* Enhanced all primary buttons with high-contrast white borders and improvedd
hover states for better readability..
Layout Polish:* Refined typography and card spacing for a more premium feel..

@EliasL-git
fix-hackclub subm
Loading
Loading status checks…
3ab2fef
@christianwell
Copy link
Copy Markdown
Member

christianwell commented May 8, 2026

The

Find your regional channel
(shares your IP with geolocation service))

dosnt work !
image
it shows me as from the usa when am from denmark!!

@EliasL-git
Copy link
Copy Markdown
Contributor Author

EliasL-git commented May 8, 2026

@christianwell You have a VPN on, it works fine for me

@christianwell
Copy link
Copy Markdown
Member

christianwell commented May 8, 2026 •
edited
Loading

i dont have a vpn on!
idk why but ur code has sum todo with npm run devv
image

but it stil dosnt work in proud mode
image

@EliasL-git
Copy link
Copy Markdown
Contributor Author

EliasL-git commented May 8, 2026

@christianwell haha some of my testing code sliped thorugh! submiting a fix

@christianwell
Copy link
Copy Markdown
Member

christianwell commented May 8, 2026

also it would simply not comple on my system! not sure if thats a me problum but its worth saying

@christianwell
Copy link
Copy Markdown
Member

christianwell commented May 8, 2026

image /api/geo

wally-ben
wally-ben suggested changes May 11, 2026
View reviewed changes
Copy link
Copy Markdown

@wally-ben wally-ben left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey Elias! Other than the Hardcoded US Fallback that Christian found, I wanna point out that geoip-lite won't work reliably on vercel. Thankfully, Vercel already provides geo data via request headers on every incoming request so there's no need for geoip-lite or any extra deoendency at all

@christianwell
Copy link
Copy Markdown
Member

christianwell commented May 11, 2026

am sure this is hosted on coolify tho

@wally-ben
Copy link
Copy Markdown

wally-ben commented May 11, 2026

That changes a lot, if it's self-hosted I have no idea why it didn't work for you 😭

@christianwell
Copy link
Copy Markdown
Member

christianwell commented May 11, 2026

that ss is just my local pc

@wally-ben
Copy link
Copy Markdown

wally-ben commented May 11, 2026

Then removing the Hardcoded Fallback should be enough?

Remove hardcoded geolocation fallback
Loading
Loading status checks…
309b89b
@EliasL-git
Copy link
Copy Markdown
Contributor Author

EliasL-git commented May 11, 2026

@wally-ben The fake location data shows up when running in dev. Also it shall now fall back to the browsers locsale when no data is available from the IP!

Use browser timezone for geolocation
Loading
Loading status checks…
7827741
@EliasL-git EliasL-git requested a review from rom wally-ben May 11, 2026 16:06
@EvanStreams EvanStreams merged commit mit 220022e into hackclub:main May 11, 2026
1 of 2 checks passed
EliasL-git added a commit to EliasL-git/slack.hackclub.com that referenced this pull request May 11, 2026
@EliasL-git
Merge pull request #1 from hackclub/main

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
GPG key ID: B5690EEEBB952194
Verified
Learn about vigilant mode
944f711 
Merge pull request hackclub#48 from EliasL-git/main
@EliasL-git EliasL-git mentioned this pull request May 11, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@wally-ben wally-ben

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

None yet

4 participants

@EliasL-git @christianwell @wally-ben @EvanStreams