Skip to content

Conversation

@zhouxl
Copy link
Collaborator

@zhouxl zhouxl commented Apr 14, 2025

Related Issue

Summary of Changes

Need Regression Testing

  • Yes
  • No

Risk Assessment

  • Low
  • Medium
  • High

Additional Notes

Screenshots (if applicable)

zhouxl and others added 19 commits April 1, 2025 06:01
…nft-tab

Feat/896 feature add new UI for nft tab
…ansactions-that-allow-paying-for-nft-bridging-fees-to-evm
…n-with-transactions-that-allow-paying-for-nft-bridging-fees-to-evm

feat: EVM Bridge Fee Payer #1016
* add: fetch blocklist

* feat: blacklist logic

* add: blocklist UI #1043
* feat: update blocklist UI  and copy,movefee: 0.0001

* feat: update copy move_fee_cost
* add: report user attribute

* feat: add private view for key

* feat:add error for send/move

* feat: remove debug info

* feat: Actor ScriptStore

* feat: remove cache of script id

* feat: update info about instabug report content

* feat: Update API to new: https://web.api.wallet.flow.com

* feat: udpate instabug
feat: udpate version instabug
# Conflicts:
#	FRW.xcodeproj/project.pbxproj
@zhouxl zhouxl requested a review from lmcmz April 14, 2025 10:15
@github-actions
Copy link

github-actions bot commented Apr 14, 2025

PR Summary

Updated repository code ownership from outblock to onflow team and modified Instabug package version requirements. The changes reflect team restructuring and dependency version adjustments for better compatibility.

Changes

File Summary
.github/CODEOWNERS Changed code ownership from @outblock/mobile-dev to @onflow/wallet-mobile, indicating a transfer of project ownership and review responsibilities.
FRW.xcodeproj/project.pbxproj Modified Instabug Swift Package version requirements, changing maximum version from 16.0.0 to 15.0.0 and minimum version from 15.0.0 to 14.2.0 for better compatibility.

autogenerated by presubmit.ai

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🚨 Pull request needs attention.

Review Summary

Commits Considered (19)
  • 6c739af: Merge branch 'main' into merge/244

Conflicts:

FRW.xcodeproj/project.pbxproj

  • 782954f: Merge pull request #1099 from onflow/change-code-ownner

Update CODEOWNERS

feat: udpate version instabug

  • add: report user attribute

  • feat: add private view for key

  • feat:add error for send/move

  • feat: remove debug info

  • feat: Actor ScriptStore

  • feat: remove cache of script id

  • feat: update info about instabug report content

  • feat: Update API to new: https://web.api.wallet.flow.com

  • feat: udpate instabug

  • 6e1a1e0: feat: update blocklist UI and copy,movefee: 0.0001 (#1074)
  • feat: update blocklist UI and copy,movefee: 0.0001

  • feat: update copy move_fee_cost

  • 3995826: fix: - switch grid -remove button of add on EVM - remove NFT fav (#1073)
  • 9974450: add: fetch blocklist (#1047)
  • add: fetch blocklist

  • feat: blacklist logic

  • add: blocklist UI #1043

  • 397561e: Merge pull request #1020 from onflow/feat/1016-feature-add-integration-with-transactions-that-allow-paying-for-nft-bridging-fees-to-evm

feat: EVM Bridge Fee Payer #1016

  • 8b54e6c: Merge branch 'develop' into feat/1016-feature-add-integration-with-transactions-that-allow-paying-for-nft-bridging-fees-to-evm
  • 43d4654: Merge pull request #1046 from onflow/fix/move-fee

fix: move fee show error

  • d82990c: Merge pull request #1052 from onflow/feat/896-feature-add-new-ui-for-nft-tab

Feat/896 feature add new UI for nft tab

Sync with Main

Files Processed (2)
  • .github/CODEOWNERS (1 hunk)
  • FRW.xcodeproj/project.pbxproj (1 hunk)
Actionable Comments (1)
  • FRW.xcodeproj/project.pbxproj [9843-9844]

    possible issue: "Package version downgrade might affect functionality"

Skipped Comments (0)

@lmcmz lmcmz merged commit 922af34 into main Apr 14, 2025
2 checks passed
@lmcmz lmcmz deleted the merge/244 branch April 14, 2025 10:16
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