Skip to content

Add missing cities for south sudan#103

Merged
ashkulz merged 2 commits into
ashkulz:masterfrom
tomi3-11:feature/add_more_cities
Jan 6, 2026
Merged

Add missing cities for south sudan#103
ashkulz merged 2 commits into
ashkulz:masterfrom
tomi3-11:feature/add_more_cities

Conversation

@tomi3-11

Copy link
Copy Markdown
Contributor

This pull request expands the search preset for South Sudan to include additional relevant keywords and locations. This will help ensure that queries using this preset capture a broader set of results related to South Sudan.

Preset improvements:

  • Expanded the include list for the "South Sudan" preset in presets.go to add "south+sudan", "yei", "wau", "aweil", "jonglei", and "maridi" alongside the existing terms.

Copilot AI review requested due to automatic review settings December 12, 2025 21:24

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This pull request enhances the South Sudan preset by expanding its search coverage to include additional significant cities and an alternative country name format. The changes align with the existing pattern of including multiple cities and regions in country presets to capture a broader set of search results.

Key changes:

  • Adds 5 new cities (Yei, Wau, Aweil, Maridi) and 1 state (Jonglei) to the South Sudan preset
  • Includes URL-encoded variant "south+sudan" alongside the existing "south sudan" term

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ashkulz

ashkulz commented Dec 15, 2025

Copy link
Copy Markdown
Owner

@tomi3-11 why did you add south+sudan when south sudan is already there?

@tomi3-11

Copy link
Copy Markdown
Contributor Author

I wanted to expand the search to eliminate Sudan if searching for " South Sudan" I think that can be eliminated.

Comment thread presets.go Outdated
@ashkulz ashkulz merged commit 8cbfaf2 into ashkulz:master Jan 6, 2026
1 check passed
@ashkulz

ashkulz commented Jan 6, 2026

Copy link
Copy Markdown
Owner

Thanks for the contribution, @tomi3-11!

@tomi3-11

tomi3-11 commented Jan 6, 2026

Copy link
Copy Markdown
Contributor Author

@ashkulz my pleasure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants