Skip to content

Zaahir Moolla
moollaza

Organizations

@duckduckgo
@moollaza
@moollaza
Whois: Fix JavaScript bug where we attempt to access an unassigned property of an object
1 commit with 1 addition and 1 deletion
moollaza commented on pull request duckduckgo/zeroclickinfo-spice#2507
@moollaza

@sekhavati Thanks! This LGTM :+1:

@moollaza
@moollaza
GTA V CheatSheet
1 commit with 119 additions and 0 deletions
@moollaza

@javathunderman Thanks! This LGTM :+1:

@moollaza
New Telugu CheatSheet
@moollaza
New Telugu CheatSheet
@moollaza

@roxxup I'm going to close these Puil Request because they're likely going to lead to problems

@moollaza

someone help i don't know what's going on @roxxup I see you have several PR's with several different files being modified. You need to separate a…

@moollaza
  • @moollaza c64f577
    Merge pull request #2189 from bigcurl/only_trigger_when_usefull_city
  • @bigcurl a071892
    Merge branch 'master' into only_trigger_when_usefull_city
  • 11 more commits »
@moollaza
Spice wgha: Reduce cache time and add user location to query
12 commits with 22 additions and 24 deletions
moollaza commented on pull request duckduckgo/zeroclickinfo-spice#2189
@moollaza

Thanks! This LGTM :+1:

@moollaza

@javathunderman sorry but you need to add 1 IA at a time -- that's why beta isn't deploying the PR. The code looks pretty good though. I would sugg…

@moollaza

I'm combining this w/ #2168 into one PR. (Hooray! Two in one! :smile:) @javathunderman whoops, that's actually not allowed. The platform ensure that yo…

@moollaza
moollaza deleted branch pr/2215 at duckduckgo/zeroclickinfo-goodies
@moollaza
Continue Pr/2215
2 commits with 313 additions and 0 deletions
moollaza commented on pull request duckduckgo/zeroclickinfo-spice#2502
@moollaza

:+1:

moollaza commented on pull request duckduckgo/zeroclickinfo-spice#2502
@moollaza

Actually, it may be best to put the list of items and their values into a table in the item_detail pane. Can you switch this to the list template a…

moollaza commented on pull request duckduckgo/zeroclickinfo-spice#2502
@moollaza

No need to prefix Dates it'll be pretty clear what we're showing is a list/range of dates. We should use Moment.js to format the dates so they're e…

@moollaza
moollaza deleted branch zaahir/pwned-indentation at duckduckgo/zeroclickinfo-spice
@moollaza
Fix indentations, switch tabs to spaces
1 commit with 41 additions and 41 deletions
@moollaza
Fix indentations, switch tabs to spaces
1 commit with 41 additions and 41 deletions
moollaza commented on pull request duckduckgo/zeroclickinfo-spice#2502
@moollaza

Let's disable the item_detail and detail views: group: 'text', detail: false, item_detail: false

moollaza commented on pull request duckduckgo/zeroclickinfo-spice#2502
@moollaza

No need to keep restating "Sales Tax Holidays" for each tile. The title should just be the state name

moollaza commented on pull request duckduckgo/zeroclickinfo-spice#2502
@moollaza

Actually, we should be able to simplify this to something like if (!(api_result && api_result.length)) and then use a relevancy block to ensure req…

moollaza commented on pull request duckduckgo/zeroclickinfo-spice#2502
@moollaza

We should only need sourceName: "SnapCX", sourceUrl: "https://snapcx.io" -- if you have a favicon we'll pick it up. Do you have an endpoint on your…

Something went wrong with that request. Please try again.