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

Cannot read property 'split' of undefined #1025

Closed
owocki opened this issue Apr 30, 2018 · 5 comments
Closed

Cannot read property 'split' of undefined #1025

owocki opened this issue Apr 30, 2018 · 5 comments

Comments

@owocki
Copy link
Contributor

owocki commented Apr 30, 2018

possibly related to #790

TypeError: Cannot read property 'split' of undefined
1
File https://gitcoin.co/static/v2/js/pages/bounty_details.61a7dfc7b1b2.js line 139 col 50 in Object.issue_keywords
2
File https://gitcoin.co/static/v2/js/pages/bounty_details.61a7dfc7b1b2.js line 472 col 31 in build_detail_page
3
File https://gitcoin.co/static/v2/js/pages/bounty_details.61a7dfc7b1b2.js line 654 col 9 in Object.success
4
File https://gitcoin.co/static/v2/js/jquery.c9f5aeeca3ad.js line 2 col 28017 in i
5
File https://gitcoin.co/static/v2/js/jquery.c9f5aeeca3ad.js line 2 col 28783 in Object.fireWith [as resolveWith]
6
File https://gitcoin.co/static/v2/js/jquery.c9f5aeeca3ad.js line 4 col 14035 in A
7
File https://gitcoin.co/static/v2/js/jquery.c9f5aeeca3ad.js line 4 col 16323 in XMLHttpRequest.<anonymous>
@owocki
Copy link
Contributor Author

owocki commented Apr 30, 2018

@eswarasai this has been showing up a lot since #790 was merged.. got a sec to take a look?

@owocki
Copy link
Contributor Author

owocki commented Apr 30, 2018

@eswarasai
Copy link
Contributor

eswarasai commented Apr 30, 2018

@owocki -- Sure. I don't have access to the Rollbar though. My guess is, there's a null check missing for already existing localStorage values.

@mbeacom
Copy link
Contributor

mbeacom commented Apr 30, 2018

@eswarasai I sent you an invitation to rollbar.

@eswarasai
Copy link
Contributor

Thanks @mbeacom for the invitation. As I can see from the above logs, the error occurs in bounty_detail.js which is left untouched in #790 as you can see in this commit.

I think I might know the root cause for this. There might be a GitHub Project for which we couldn't retrieve any Keyword. If it's easy, try looking up in the DB to see which Bounty.metadata has issueKeywords as empty string.

@owocki owocki closed this as completed in ec84c17 Apr 30, 2018
CuriousLearner added a commit to CuriousLearner/web that referenced this issue May 1, 2018
…6-296

* 'master' of https://github.com/gitcoinco/web: (31 commits)
  fixes https://rollbar.com/gitcoin/gitcoin/items/908
  dashboard.js -- onboard search was expecing falsy value, not -1
  measure hourly rates of different repos
  Fix url collision
  Restrict onboarding view to staff temporarily
  sets default search project status to 'any' (gitcoinco#936)
  staggers cron start times (gitcoinco#1034)
  dont show email address if not auth
  fixes gitcoinco#1025
  obfuscate usernames
  review: added feedback
  review: added feedback
  onboard: refactored code
  dashboard: handle redirect from onboard
  ftux: birth
  WIP: Update FaucetRequest with FK to Profile (gitcoinco#884)
  fixes tests
  no more underscore methods for non-private methods
  Fix gitcoinco#1032 - Resolve trans rendering for faucet request copy
  allow user to skip waiting for tip tx to clear blockchain
  ...
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

No branches or pull requests

3 participants