Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Add assignCreditToUploader option when pushing #24

Closed
wants to merge 2 commits into from

Conversation

alex-sl-eng
Copy link
Member

https://bugzilla.redhat.com/show_bug.cgi?id=1184299

See also:
Server: zanata/zanata-server#678
Client: zanata/zanata-client#46

DO NOT MERGE UNTIL Zanata-Server implementation is done.

@alex-sl-eng alex-sl-eng force-pushed the rhbz1184299 branch 2 times, most recently from b33e4b4 to b1f2dd5 Compare January 28, 2015 01:03
@@ -50,6 +50,14 @@ public Person getTranslator() {
return translator;
}

@Deprecated
/**
* Zanata server will ignore this field if set,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not remember this being part of the criteria we decided, so I looked for the criteria and could not find any. Did we really decide to make the translator field not work?

@aeng What criteria are you using to implement this?
@djansen-redhat What criteria will you be using to verify this?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We never used this field at all. To simplify implementation on server, this should be deprecated.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, if we never used it then it is fine to deprecate.

@davidmason
Copy link
Contributor

👍

@alex-sl-eng alex-sl-eng changed the title Add isMyTranslations option when pushing: https://bugzilla.redhat.com/sh... Add assignCreditToUploader option when pushing: https://bugzilla.redhat.com/sh... Jan 28, 2015
@alex-sl-eng alex-sl-eng changed the title Add assignCreditToUploader option when pushing: https://bugzilla.redhat.com/sh... Add assignCreditToUploader option when pushing Jan 28, 2015
@alex-sl-eng
Copy link
Member Author

Merged into integration/master. 86e0008

@alex-sl-eng
Copy link
Member Author

Merged into integration/master. bb9f2c7

@alex-sl-eng alex-sl-eng closed this Feb 5, 2015
@alex-sl-eng alex-sl-eng deleted the rhbz1184299 branch February 5, 2015 05:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants