`undefined (Issue)` card is created when I use version 0.7

Question

shige opened this issue 7 years ago · comments

Hi, Thank you for updating the bug2trello chrome extension.
But I found the bug.

If I clicked the Add to Trello button, the card is created with title com: #25099 - undefined (Issue).
It seems lacking the pull request title.

Could you fix this bug if you can?

I'm using version 0.7.

Thanks!

Andrew Starr-Bochicchio · Answer 1 · Thu May 11 2017 00:04:20 GMT+0800 (China Standard Time)

Thanks for the report. What platform is the bug/issue on?

Dmitry Parshenko · Answer 2 · Mon May 22 2017 15:53:02 GMT+0800 (China Standard Time)

Encountered the same issue with Bitbucket.

Tadashi Shigeoka · Answer 3 · Tue May 23 2017 17:10:25 GMT+0800 (China Standard Time)

What platform is the bug/issue on?

the info is below.

Tadashi Shigeoka · Answer 4 · Fri Jun 09 2017 11:29:25 GMT+0800 (China Standard Time)

I check-outed the old version 0.6 and I'm using it now.

Edouard Oger · Answer 5 · Tue Jul 25 2017 10:08:29 GMT+0800 (China Standard Time)

I just committed a fix (ae9a8e7) for that issue on master, could you try and see if this resolves the problem for you @bakorer @kongo?

Tadashi Shigeoka · Answer 6 · Thu Jul 27 2017 02:13:54 GMT+0800 (China Standard Time)

@eoger CC: @kongo Thank you for fixing this problem. 👏

I checked on the current master branch ( ae9a8e7 ) , but it seems some errors on chrome developer tools. 👀

Could you check these errors ❓

index.js Failed to load resource: net::ERR_FILE_NOT_FOUND
client.js:11 Uncaught ReferenceError: jQuery is not defined
    at client.js:11

Edouard Oger · Answer 7 · Thu Jul 27 2017 02:20:10 GMT+0800 (China Standard Time)

I had the same problem, it seems that npm run dev is broken. Run npm run build and load the temporary addon from dist/

Tadashi Shigeoka · Answer 8 · Thu Jul 27 2017 02:24:07 GMT+0800 (China Standard Time)

@eoger Thanks!
Which Node.js version should I use?

Edouard Oger · Answer 9 · Thu Jul 27 2017 02:26:13 GMT+0800 (China Standard Time)

I think as long as it's recent it's fine. I used node 8 to work on that patch.

Tadashi Shigeoka · Answer 10 · Thu Jul 27 2017 02:35:27 GMT+0800 (China Standard Time)

@eoger It works on GitHub private issue and pull-request. 👏

📝 Tried following

nvm install v8.2.1
npm install
npm run build

Load dist dir via Chrome browser.

Edouard Oger · Answer 11 · Thu Jul 27 2017 02:37:38 GMT+0800 (China Standard Time)

Awesome, thank you for testing!

Tadashi Shigeoka · Answer 12 · Mon Jun 10 2019 20:11:24 GMT+0800 (China Standard Time)

It seems fixed on current master ae9a8e7