Adding Bitbucket support. #94

Merged
merged 1 commit into from Jul 13, 2012

Conversation

Projects
None yet
4 participants
@AlmogBaku
Contributor

AlmogBaku commented Jul 11, 2012

fix issue #1128

Adding Bitbucket support.
fix issue #1128

** Changes:
- changing all the translations strings
- adding documentation
- fixing browser bug
@AlmogBaku

This comment has been minimized.

Show comment
Hide comment

zegenie added a commit that referenced this pull request Jul 13, 2012

Merge pull request #94 from AlmogBaku/next
Adding Bitbucket support.

@zegenie zegenie merged commit f37d639 into thebuggenie:next Jul 13, 2012

@Grexeo

This comment has been minimized.

Show comment
Hide comment
@Grexeo

Grexeo Feb 20, 2013

Contributor

Discovered a really annoying typo after trying to set this up - the doc (modules/publish/fixtures/VCSIntegration) tells you to set the bitbucket URL to:

http://www.your-tbg-site.com/vcs_integration/report/PROJECTID/bitcucket/?passkey=PASSKEY

Obviously it should be bitbucket, not bitcucket. I copy + pasted the URL and then adapted it, so it took me a while to find the problem.

Contributor

Grexeo commented Feb 20, 2013

Discovered a really annoying typo after trying to set this up - the doc (modules/publish/fixtures/VCSIntegration) tells you to set the bitbucket URL to:

http://www.your-tbg-site.com/vcs_integration/report/PROJECTID/bitcucket/?passkey=PASSKEY

Obviously it should be bitbucket, not bitcucket. I copy + pasted the URL and then adapted it, so it took me a while to find the problem.

@AlmogBaku

This comment has been minimized.

Show comment
Hide comment
@AlmogBaku

AlmogBaku Feb 20, 2013

Contributor

it seems you right! I'll work on it ASAP.

please open an issue..

Contributor

AlmogBaku commented Feb 20, 2013

it seems you right! I'll work on it ASAP.

please open an issue..

@Grexeo

This comment has been minimized.

Show comment
Hide comment
@Grexeo

Grexeo Feb 20, 2013

Contributor

Already done, and it seems someone beat me to it ;)

Contributor

Grexeo commented Feb 20, 2013

Already done, and it seems someone beat me to it ;)

@AlmogBaku

This comment has been minimized.

Show comment
Hide comment
@AlmogBaku

AlmogBaku Feb 20, 2013

Contributor

can you please attach the issue ID/link?

Contributor

AlmogBaku commented Feb 20, 2013

can you please attach the issue ID/link?

@Grexeo

This comment has been minimized.

Show comment
Hide comment
@Grexeo

Grexeo Feb 20, 2013

Contributor

Certainly: http://issues.thebuggenie.com/thebuggenie/issues/1926

And here's the duplicate I opened that needs deleting/closing: http://issues.thebuggenie.com/thebuggenie/issues/1947 (Sorry about that!)

Contributor

Grexeo commented Feb 20, 2013

Certainly: http://issues.thebuggenie.com/thebuggenie/issues/1926

And here's the duplicate I opened that needs deleting/closing: http://issues.thebuggenie.com/thebuggenie/issues/1947 (Sorry about that!)

@sharrondenice

This comment has been minimized.

Show comment
Hide comment
@sharrondenice

sharrondenice Feb 21, 2013

Contributor

Has work been completed for this bug? Neither ticket has any code changes I see...are we missing a commit?

Contributor

sharrondenice commented Feb 21, 2013

Has work been completed for this bug? Neither ticket has any code changes I see...are we missing a commit?

@sharrondenice

This comment has been minimized.

Show comment
Hide comment
@sharrondenice

sharrondenice Feb 21, 2013

Contributor

I added in the fix I think in the confusion I think you all may have thought the duplicate issue had been worked but it hadn't...I added in the fix and it'll be pulled in soon.

Contributor

sharrondenice commented Feb 21, 2013

I added in the fix I think in the confusion I think you all may have thought the duplicate issue had been worked but it hadn't...I added in the fix and it'll be pulled in soon.

@AlmogBaku

This comment has been minimized.

Show comment
Hide comment
@AlmogBaku

AlmogBaku Feb 21, 2013

Contributor

great! thanks!
and sorry for the confusing bug :(

Contributor

AlmogBaku commented Feb 21, 2013

great! thanks!
and sorry for the confusing bug :(

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