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

[feat][plugin]add offchain-plugin for chainmaker(chainable) #50

Merged

Conversation

ccsols
Copy link
Contributor

@ccsols ccsols commented Jul 8, 2024

贡献者:
上海浦江数链数字科技有限公司
长三角数链(上海)网络基础设施有限公司

@bladechen
Copy link
Collaborator

有相关使用与设计的README么?

@ccsols ccsols force-pushed the feat/chainmaker-chainable branch from bf03b5a to 105afd5 Compare July 9, 2024 02:22
@ccsols
Copy link
Contributor Author

ccsols commented Jul 9, 2024

有相关使用与设计的README么?

已添加README

@zouxyan zouxyan requested a review from dawn-to-dusk July 10, 2024 09:22
@zouxyan zouxyan added the bbc-plugin Contributions about BBC plugins label Jul 10, 2024
@dawn-to-dusk
Copy link
Collaborator

  1. 首先如果系统合约是需要提前部署的话希望在readme里面说明一下怎么使用哈,其次代码里好像没有看到系统合约地址注入的地方,bbccontext里面的合约地址是什么时候注入的呀?
  2. 这个插件在哪个版本的chainmaker上测试通过了也在readme里面说明一下哈(建议同时说明测试通过的链版本和链sdk版本)

@ccsols
Copy link
Contributor Author

ccsols commented Oct 14, 2024

  1. 首先如果系统合约是需要提前部署的话希望在readme里面说明一下怎么使用哈,其次代码里好像没有看到系统合约地址注入的地方,bbccontext里面的合约地址是什么时候注入的呀?
  2. 这个插件在哪个版本的chainmaker上测试通过了也在readme里面说明一下哈(建议同时说明测试通过的链版本和链sdk版本)

谢谢回复!已经在readme添加了提前部署系统合约的操作步骤,包括支持的链和sdk的版本

@dawn-to-dusk
Copy link
Collaborator

dawn-to-dusk commented Oct 14, 2024 via email

@ccsols
Copy link
Contributor Author

ccsols commented Oct 24, 2024

邮件已收到,谢谢啦~

你好,请问还有没有什么问题 :)

@Override
public void setupAuthMessageContract() {
AuthMessageContract am = new AuthMessageContract();
am.setContractAddress(sdk.getAmContractAddress));
Copy link
Collaborator

Choose a reason for hiding this comment

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

getAmContractAddress 这个是个方法还是参数呀?这里好像有报错编译不了?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

是方法,谢谢提醒,已经修复了

@dawn-to-dusk dawn-to-dusk merged commit aed9024 into AntChainOpenLabs:main Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bbc-plugin Contributions about BBC plugins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants