Skip to content

Added pushing citations to Texmaker#204

Merged
koppor merged 1 commit into
JabRef:masterfrom
oscargus:texmaker
Oct 4, 2015
Merged

Added pushing citations to Texmaker#204
koppor merged 1 commit into
JabRef:masterfrom
oscargus:texmaker

Conversation

@oscargus

@oscargus oscargus commented Oct 2, 2015

Copy link
Copy Markdown
Contributor

Implements SourceForge feature requests #318 and #582.

@koppor

koppor commented Oct 4, 2015

Copy link
Copy Markdown
Member

Looks good. Minor comments above. The abstract class AbstractPushToApplication could be left as future work. 😃

@oscargus

oscargus commented Oct 4, 2015

Copy link
Copy Markdown
Contributor Author

Thanks.

I agree. There are changes to the other PushToXXX classes in the FormBuilder pull request, so I would prefer to base the AbstractPushToApplication on the unified source. :-) (Especially with my still improvable git wizardry).

@koppor

koppor commented Oct 4, 2015

Copy link
Copy Markdown
Member

OK, I let this go through, we will work on #205 and possibly finally with another PR.

koppor added a commit that referenced this pull request Oct 4, 2015
Added pushing citations to Texmaker
@koppor koppor merged commit b9ff425 into JabRef:master Oct 4, 2015
@oscargus oscargus deleted the texmaker branch October 4, 2015 22:34
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

Successfully merging this pull request may close these issues.

2 participants