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.

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

Maryia/webrel-483/TS migration of ContractType function #17

Conversation

maryia-deriv
Copy link
Collaborator

@maryia-deriv maryia-deriv commented Jul 18, 2023

Changes:

  • TS migration of ContractType self-invoking function.

Screenshots:

Please provide some screenshots of the change.

@maryia-deriv maryia-deriv marked this pull request as ready for review July 19, 2023 16:00
@maryia-deriv maryia-deriv changed the title Maryia/webrel-483/TS migration of contract-type Maryia/webrel-483/TS migration of ContractType function Jul 19, 2023
@coveralls
Copy link

coveralls commented Jul 19, 2023

Pull Request Test Coverage Report for Build 5609287870

  • 1 of 82 (1.22%) changed or added relevant lines in 2 files are covered.
  • 5 unchanged lines in 5 files lost coverage.
  • Overall coverage decreased (-0.002%) to 8.911%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/shared/src/utils/helpers/start-date.ts 1 3 33.33%
packages/trader/src/Stores/Modules/Trading/Helpers/contract-type.ts 0 79 0.0%
Files with Coverage Reduction New Missed Lines %
packages/p2p/src/constants/api-error-codes.js 1 50.0%
packages/p2p/src/constants/buy-sell.js 1 50.0%
packages/p2p/src/constants/floating-rate.js 1 50.0%
packages/p2p/src/constants/my-profile-tabs.js 1 50.0%
packages/p2p/src/constants/order-list.js 1 50.0%
Totals Coverage Status
Change from base Build 5505818751: -0.002%
Covered Lines: 8517
Relevant Lines: 79369

💛 - Coveralls

@github-actions
Copy link

github-actions bot commented Jul 19, 2023

Generating Lighthouse report...

Copy link

@henry-deriv henry-deriv left a comment

Choose a reason for hiding this comment

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

lgtm

@kate-deriv kate-deriv merged commit fd8882c into kate-deriv:kate/dtra-269/ts_migration_trader_package_2 Jul 25, 2023
1 check passed
kate-deriv added a commit that referenced this pull request Sep 27, 2023
…ckage (binary-com#9284)

* fix: ts-migrate screen small

* fix: sass was unable to be found

* chore: add key files from prev branch

* chore: empty commit

* Maryia/dtra-279/TS migration [Trader]: TradeModals, MarketUnavailableModal & UnsupportedContractModal (#16)

* maryia/WEBREL-323/feat: ts migration of base-store (#18)

* chore: start ts migration of base-store

* chore: add more types to BaseStore

* chore: added more types to base-store

* chore: added more types to base-store

* chore: finalize base-store ts migration

* chore: reorder imports

* Maryia/webrel-483/TS migration of ContractType function (#17)

* chore: start ts migration

* chore: continue ts migration of contract-types

* chore: add more types to contract-types

* chore: add type for getContractValues return value

* chore: improve reduce types

* fix: sonarcloud issues (#19)

* chore: update reg exp

* chore: remove code smell

* chore: remove security hotspot

* chore: update reg exp

* chore: update reg exp to prev version

* fix: conflict

* fix: ts error

* refactor: apply suggestions

* fix: resolve more conflicts

* fix: conflicts in start date

* fix: type of onchangestartdate

* fix: type of contract cat list in usetraderstore

* fix: types in contract type and allow equals

* refactor: aplly part of suggestions

* refactor: remove unused type

* refactor: remove wrong types

* refactor: types of time

* refactor: revert changes

* chore: empty commit

* fix: more conflicts

---------

Co-authored-by: Henry Hein <henry@regentmarkets.com>
Co-authored-by: Akmal Djumakhodjaev <akmal@binary.com>
Co-authored-by: Maryia <103177211+maryia-deriv@users.noreply.github.com>
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.

6 participants