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.

Sign up for GitHub

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

Jump to bottom

Make Exchange-specific DTOs Immutatble #2

Closed
timmolter opened this issue Jun 16, 2012 · 2 comments
Closed

Make Exchange-specific DTOs Immutatble #2

timmolter opened this issue Jun 16, 2012 · 2 comments
Assignees
@timmolter

Comments

@timmolter
Copy link
Member

... by using a nice feature from Jackson: Use any constructors, factory methods that you want (not just default zero-arg one!). See http://www.cowtowncoder.com/blog/archives/2010/11/entry_434.html
@ghost ghost assigned timmolter Jun 16, 2012
@efinery
Copy link

efinery commented Jun 18, 2012

Hi Tim,

Hope all is well. Very nice feature, I also really like the Materialized Interfaces. As a colleague once told me, "the best line of code is the one you don't have to write" :-)

Hugh

@gary-rowe
Copy link
Collaborator

Tim, this is definitely worth doing.

timmolter added a commit that referenced this issue Sep 16, 2012
@timmolter
Implemented Issue #2: Make Exchange-specific DTOs Immutatble #2
843e45f
@timmolter timmolter mentioned this issue Jan 26, 2013
Closed
@sideburnie sideburnie mentioned this issue Apr 6, 2013
Closed
timmolter pushed a commit that referenced this issue Mar 7, 2014
@ObsessiveOrange
Merge pull request #2 from timmolter/develop
91d57a2 
Merge changes from main repo
timmolter pushed a commit that referenced this issue May 31, 2014
@ObsessiveOrange
Merge pull request #2 from timmolter/develop
70aa95c 
Merge changes
@jpe42 jpe42 mentioned this issue Jun 28, 2014
Closed
timmolter pushed a commit that referenced this issue Mar 26, 2015
@glabmoris
Merge pull request #2 from timmolter/develop
096d02f 
fork update
@kryptohash kryptohash mentioned this issue May 30, 2015
Closed
timmolter pushed a commit that referenced this issue Jun 8, 2015
@bryan4887
Merge pull request #2 from bryan4887/develop-bitcoindeAdjustments
8d3c257 
Develop bitcoinde adjustments
timmolter pushed a commit that referenced this issue Jul 13, 2015
@stachon
Merge pull request #2 from timmolter/develop
e1e2193 
Merge from upstream
@baffo32 baffo32 mentioned this issue Nov 26, 2015
Closed
timmolter pushed a commit that referenced this issue May 16, 2017
@Mikelle
Merge pull request #2 from timmolter/develop
1cd517e 
merge
timmolter pushed a commit that referenced this issue Oct 25, 2017
@allenday
Merge pull request #2 from hardsky/develop
3a22bf9 
Add BCC currency
timmolter pushed a commit that referenced this issue Feb 6, 2018
@makarid
Merge pull request #2 from timmolter/develop
0dd83e5 
Merging
timmolter pushed a commit that referenced this issue Mar 1, 2018
@cacofishadct
Merge pull request #2 from timmolter/develop
443bfa9 
Merge
timmolter pushed a commit that referenced this issue May 23, 2018
@trixprod
Merge pull request #2 from knowm/develop
afd46ed 
Pull request
timmolter pushed a commit that referenced this issue Jun 10, 2018
@makarid
Merge pull request #2 from knowm/develop
6f9f30a 
sad
@traviscollins traviscollins mentioned this issue Jul 9, 2018
Closed
timmolter pushed a commit that referenced this issue Oct 27, 2018
@dsklyadnev
Merge pull request #2 from knowm/develop
caa1952 
Merge pull request #2830 from mixtocoin/develop
timmolter pushed a commit that referenced this issue Mar 8, 2019
@makarid
Merge pull request #2 from knowm/develop
03f07ff 
update fork
timmolter pushed a commit that referenced this issue Apr 7, 2019
@makarid
Merge pull request #2 from knowm/develop
b376a1f 
Update project
timmolter pushed a commit that referenced this issue Jun 11, 2019
@gregoreficint
Merge pull request #2 from knowm/develop
c5436fd 
Updates
timmolter pushed a commit that referenced this issue Sep 25, 2019
@vitaha93
Merge pull request #2 from knowm/develop
42fb928 
new versions pulling
timmolter pushed a commit that referenced this issue Mar 31, 2020
@lucaforni
Merge pull request #2 from knowm/develop
6a22dc4 
Merge knowm XChange develop branch
badgerwithagun pushed a commit to badgerwithagun/XChange that referenced this issue May 5, 2020
@mdvx
[Kraken] review fix knowm#2
700c7b6
timmolter pushed a commit that referenced this issue May 21, 2020
@mdvx
Merge pull request #2 from knowm/develop
e18c16c 
Merge
badgerwithagun pushed a commit that referenced this issue May 25, 2020
@earce
Merge pull request #2 from knowm/develop
1c4f174 
updating fork
timmolter pushed a commit that referenced this issue Sep 10, 2020
@alwinc
Merge pull request #2 from knowm/master
86bd025 
Master
earce pushed a commit that referenced this issue May 31, 2021
@maghteriton
Merge pull request #2 from maghteriton/poloniex-currency_data
9e6b9cd 
Fix for poloniex currency metadata information
earce pushed a commit that referenced this issue Nov 26, 2021
@mdvx
Merge pull request #2 from knowm/develop
f604f3e 
Merge
timmolter pushed a commit that referenced this issue Apr 20, 2024
@jolind93
Merge pull request #2 from Safello/TECH-2111-sol-withdrawal
9c0d99c 
TECH-2111 Bitstamp generic crypto withdrawal api method
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@timmolter timmolter

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@gary-rowe @timmolter @efinery