Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Repeated issues multiple times in app list #259

Open
iteles opened this issue Dec 22, 2017 · 2 comments
Open

Repeated issues multiple times in app list #259

iteles opened this issue Dec 22, 2017 · 2 comments

Comments

@iteles
Copy link
Member

iteles commented Dec 22, 2017

I can see multiple repetitions of the same issue in the issue list on the mini app here:
https://tudo-app.herokuapp.com/?rummage[paginate][max_page]=44&rummage[paginate][page]=1&rummage[paginate][per_page]=20&rummage[paginate][total_count]=874&rummage[search][labels][search_term][]=&rummage[search][labels][search_type]=gt&rummage[sort][field]=gh_updated_at.desc

(what an insane URL by the way)
An example is:
screen shot 2017-12-22 at 10 11 37

@samhstn
Copy link
Member

samhstn commented Dec 22, 2017

Strange, @iteles. Do you by any chance remember what you did to make those particular issues duplicate? I can't seem to get other issues to duplicate.

One solution though would be:

|> unique_constraint(:url)

Then a bit of case handling for weather urls are unique or not after each |> Issue.changeset in our hook controller
(1h)

  • Run a script through the database ensuring to remove any duplicates (unique based on url)
    (30m)
  • Ensure all tests are working as well as manual testing
    (30m)

@samhstn samhstn added the PT2h label Dec 22, 2017
@iteles
Copy link
Member Author

iteles commented Dec 22, 2017

@Shouston3 Yes! I just opened the app and clicked 'Updated' twice because of #260 and this was on the first page.

However, if you just open the app and go to the second page, you'll immediately see this on the second page (this issue by the way is already closed!):
screen shot 2017-12-22 at 16 51 43

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

No branches or pull requests

2 participants