Skip to content

Conversation

@unchangedraman
Copy link

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates

Description

Additional Changes

  • This PR modifies the database schema (database migration added)
  • This PR modifies dhall configs/environment variables

Motivation and Context

How did you test it?

Checklist

  • I formatted the code and addressed linter errors ./dev/format-all-files.sh
  • I reviewed submitted code
  • I added unit tests for my changes where possible
  • I added a CHANGELOG entry if applicable

callbackUrl = Just $ showBaseUrl cfg.callbackUrl -- ye i think we should ask for this.
}

checksum = PaytmEDC.buildChecksum merchantKeyDecrypted saleBody
Copy link
Member

Choose a reason for hiding this comment

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

instead of this we will have a checksum api that they will give us we have to call that.

import Kernel.Prelude

-- | PaytmEDC Configuration
data PaytmEDCCfg = PaytmEDCCfg
Copy link
Member

Choose a reason for hiding this comment

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

in this we have encrypted field so we have to decrypt them before using so make changes according to that

clientIdDecrypted <- decrypt cfg.clientId

let terminalId = req.metadataGatewayReferenceId -- TID passed via metadata
amountInPaise = round (req.amount * 100) :: Int
Copy link
Member

Choose a reason for hiding this comment

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

i don't think they take it in paise they take it in rupees i guess will check tomorrow though

BaseUrl ->
GenerateChecksumReq ->
m GenerateChecksumResp
generateChecksum url req = do
Copy link
Member

Choose a reason for hiding this comment

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

you are sending baseUrl but we have to take one more input in cfg called callbackurl

clientId :: EncryptedField 'AsEncrypted Text, -- Client ID for checksum generation
merchantKey :: EncryptedField 'AsEncrypted Text, -- Secret key (not used for local checksum anymore but might be needed)
baseUrl :: BaseUrl, -- API endpoint
callbackUrl :: BaseUrl -- Webhook callback URL
Copy link
Member

Choose a reason for hiding this comment

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

this we are not using

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.

3 participants