search

shinglyu / moztrap-new-ui

A new frontend UI for MozTrap that levarages on its REST API
10 stars 13 forks source link

To fix issue #135 #136

Closed hitripod closed 9 years ago

shinglyu commented 9 years ago

Hmm... I tried your patch. Here is an issue:

  1. Search product:"Firefox OS" suite:"APP USAGE", you should see 4 pages of cases
  2. Click page 4
  3. Click page 1

Expected: Goes back to page 1

Actual Got "Can't find anything. Try loosen the search criteria.", The URL it tries to reach is https://moztrap.mozilla.orgnull/ Obviously something is wrong when formating the URL

shinglyu commented 9 years ago

I see you pulled my commits into this too. Probably you used git rebase incorrectly. Could you help me by:

  1. Create a separate PR for the remove alert commit, so I can merge that first (perhaps creating a new branch based on my latest master)
  2. Fix the bug mentioned above, and create a new PR for that

Thanks!