Skip to content

Conversation

@oshchepkov
Copy link

3.3.0
Moved the following files to "src/services" folder:

alias.h / alias.cpp
offer.h / offer.cpp
cert.h / cert.cpp
ranges.h / ranges.cpp
graph.h / graph.cpp
asset.h / asset.cpp
assetallocation.h / assetallocation.cpp
escrow.h / escrow.cpp

Unit/integration tests passed:
image

@oshchepkov oshchepkov requested review from jnessbcf, radubcf and willyko and removed request for dwasyluk September 28, 2018 06:53
Copy link

@radubcf radubcf left a comment

Choose a reason for hiding this comment

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

ACK 

@sidhujag sidhujag merged commit d999ce9 into dev-3.x-prep-3.3.0 Sep 28, 2018
@oshchepkov oshchepkov deleted the SYS-324-Move-syscoin-files-into-services-folder-(3.0.3-prep) branch September 28, 2018 17:04
sidhujag pushed a commit that referenced this pull request Apr 3, 2022
… syntax for numbers with a unit symbol"

0c64401 Revert "qt: Do not use QObject::tr plural syntax for numbers with a unit symbol" (Luke Dashjr)

Pull request description:

  Apparently this got forgotten. Maybe too late for 23.x (it's a bugfix, but changes translation strings).

  This reverts commit 3adde72 (#296)

  per [GChuf](bitcoin-core/gui#296 (comment))

  >I can confirm for slovenian and other slavic languages that we do have 3 or 4 different ways of saying "%n GB needed%, depending on the actual number of gigabytes. Similar to english "is/are". There's no way to cover all cases ... this is exactly why transifex allows you to have more than 2 options.

ACKs for top commit:
  hebasto:
    ACK 0c64401, I have reviewed the code and it looks OK, I agree it can be merged.

Tree-SHA512: c01bae44a32b3ec324f2f9b8e4923bbb2e83bbd1460b745c5c911b98a9b2806fcbf815cfb19a1f1a7038c5c14312e102e7df8744c9002ef784b36d158e08eb14
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.

4 participants