Skip to content

Commit

Permalink
BCF-3234 remove app shortcut (#13353)
Browse files Browse the repository at this point in the history
* Add files with updates links.

* Update shortcut links.

* Add tag.
  • Loading branch information
pavel-raykov committed Jun 3, 2024
1 parent 02c6428 commit 7a86103
Show file tree
Hide file tree
Showing 11 changed files with 15 additions and 10 deletions.
5 changes: 5 additions & 0 deletions .changeset/neat-zebras-fix.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"chainlink": minor
---

#updated Remove deprecated app.shortcut links
2 changes: 1 addition & 1 deletion contracts/test/v0.8/automation/CronUpkeep.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ describe('CronUpkeep', () => {
it('has a limited public ABI [ @skip-coverage ]', () => {
// Casting cron is necessary due to a tricky versioning mismatch issue, likely between ethers
// and typechain. Remove once the version issue is resolved.
// https://app.shortcut.com/chainlinklabs/story/21905/remove-contract-cast-in-cronupkeep-test-ts
// https://smartcontract-it.atlassian.net/browse/ARCHIVE-22094
h.publicAbi(cron as unknown as Contract, [
's_maxJobs',
'performUpkeep',
Expand Down
2 changes: 1 addition & 1 deletion core/chains/evm/client/errors.go
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ var erigon = ClientErrors{
var arbitrumFatal = regexp.MustCompile(`(: |^)(invalid message format|forbidden sender address)$|(: |^)(execution reverted)(:|$)`)
var arbitrum = ClientErrors{
// TODO: Arbitrum returns this in case of low or high nonce. Update this when Arbitrum fix it
// https://app.shortcut.com/chainlinklabs/story/16801/add-full-support-for-incorrect-nonce-on-arbitrum
// Archived ticket: story/16801/add-full-support-for-incorrect-nonce-on-arbitrum
NonceTooLow: regexp.MustCompile(`(: |^)invalid transaction nonce$|(: |^)nonce too low(:|$)`),
NonceTooHigh: regexp.MustCompile(`(: |^)nonce too high(:|$)`),
TerminallyUnderpriced: regexp.MustCompile(`(: |^)gas price too low$`),
Expand Down
2 changes: 1 addition & 1 deletion core/chains/evm/config/toml/defaults/Ethereum_Kovan.toml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ OperatorFactoryAddress = '0x8007e24251b1D2Fc518Eb843A701d9cD21fe0aA3'

[GasEstimator]
# FIXME: Kovan has strange behaviour with EIP1559, see:
# https://app.shortcut.com/chainlinklabs/story/34098/kovan-can-emit-blocks-that-violate-assumptions-in-block-history-estimator
# https://smartcontract-it.atlassian.net/browse/BCF-1420
EIP1559DynamicFees = false

[GasEstimator.BlockHistory]
Expand Down
2 changes: 1 addition & 1 deletion core/chains/evm/config/toml/defaults/Ethereum_Rinkeby.toml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ MinContractPayment = '0.1 link'

[GasEstimator]
# TODO: EIP1559 on rinkeby has not been adequately tested, see:
# https://app.shortcut.com/chainlinklabs/story/34098/kovan-can-emit-blocks-that-violate-assumptions-in-block-history-estimator
# https://smartcontract-it.atlassian.net/browse/BCF-1420
EIP1559DynamicFees = false

[GasEstimator.BlockHistory]
Expand Down
2 changes: 1 addition & 1 deletion core/chains/evm/forwarders/forwarder_manager.go
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ type FwdMgr struct {
logpoller evmlogpoller.LogPoller

// TODO(samhassan): sendersCache should be an LRU capped cache
// https://app.shortcut.com/chainlinklabs/story/37884/forwarder-manager-uses-lru-for-caching-dest-addresses
// https://smartcontract-it.atlassian.net/browse/ARCHIVE-22505
sendersCache map[common.Address][]common.Address
latestBlock int64

Expand Down
2 changes: 1 addition & 1 deletion core/services/keeper/upkeep_executer_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -275,7 +275,7 @@ func Test_UpkeepExecuter_PerformsUpkeep_Happy(t *testing.T) {
head := newHead()
executer.OnNewLongestChain(testutils.Context(t), &head)
// TODO we want to see an errored run result once this is completed
// https://app.shortcut.com/chainlinklabs/story/25397/remove-failearly-flag-from-eth-call-task
// https://smartcontract-it.atlassian.net/browse/ARCHIVE-22186
cltest.AssertPipelineRunsStays(t, jb.PipelineSpecID, db, 0)
})

Expand Down
2 changes: 1 addition & 1 deletion core/services/ocr2/plugins/functions/reporting.go
Original file line number Diff line number Diff line change
Expand Up @@ -337,7 +337,7 @@ func (r *functionsReporting) Report(ctx context.Context, ts types.ReportTimestam
}

// TODO: support per-request aggregation method
// https://app.shortcut.com/chainlinklabs/story/57701/per-request-plugin-config
// https://smartcontract-it.atlassian.net/browse/FUN-159
aggregated, errAgg := Aggregate(defaultAggMethod, observations)
if errAgg != nil {
r.logger.Error("FunctionsReporting Report: error when aggregating reqId", commontypes.LogFields{
Expand Down
2 changes: 1 addition & 1 deletion core/services/ocr2/validate/validate.go
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ func validateSpec(ctx context.Context, tree *toml.Tree, spec job.Job, rc plugins
case types.OCR2Keeper:
return validateOCR2KeeperSpec(spec.OCR2OracleSpec.PluginConfig)
case types.Functions:
// TODO validator for DR-OCR spec: https://app.shortcut.com/chainlinklabs/story/54054/ocr-plugin-for-directrequest-ocr
// TODO validator for DR-OCR spec: https://smartcontract-it.atlassian.net/browse/FUN-112
return nil
case types.Mercury:
return validateOCR2MercurySpec(spec.OCR2OracleSpec.PluginConfig, *spec.OCR2OracleSpec.FeedID)
Expand Down
2 changes: 1 addition & 1 deletion core/services/pipeline/scheduler.go
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ func (s *scheduler) Run() {
result := <-s.resultCh
// TODO: if for some reason the cleanup didn't succeed and we're stuck waiting for reports forever
// we should be able to timeout and finish shutting down
// See: https://app.shortcut.com/chainlinklabs/story/21225/straighten-out-and-clarify-context-usage-in-the-pipeline
// See: https://smartcontract-it.atlassian.net/browse/BCF-994

s.waiting--

Expand Down
2 changes: 1 addition & 1 deletion core/web/router.go
Original file line number Diff line number Diff line change
Expand Up @@ -415,7 +415,7 @@ func v2Routes(app chainlink.Application, r *gin.RouterGroup) {
{"cosmos", NewCosmosNodesController(app)},
} {
if chain.path == "evm" {
// TODO still EVM only https://app.shortcut.com/chainlinklabs/story/26276/multi-chain-type-ui-node-chain-configuration
// TODO still EVM only . Archive ticket: story/26276/multi-chain-type-ui-node-chain-configuration
nodes.GET("", paginatedRequest(chain.nc.Index))
}
nodes.GET(chain.path, paginatedRequest(chain.nc.Index))
Expand Down

0 comments on commit 7a86103

Please sign in to comment.