Skip to content

[venus-gateway] 删除gateway中api包的间接引用，在代码中直接使用shared中的接口 #4895

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.

Sign up for GitHub

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

Jump to bottom
Closed
5 of 17 tasks
Tracked by #4909
hunjixin opened this issue May 26, 2022 · 0 comments · Fixed by ipfs-force-community/sophon-gateway#56
Closed
5 of 17 tasks
Tracked by #4909

[venus-gateway] 删除gateway中api包的间接引用，在代码中直接使用shared中的接口 #4895

hunjixin opened this issue May 26, 2022 · 0 comments · Fixed by ipfs-force-community/sophon-gateway#56
Assignees
@hunjixin @LinZexiao
Labels
CU-chain-service Venus chain service related issues V-venus-gateway venus gateway component

Comments

@hunjixin
Copy link
Contributor

hunjixin commented May 26, 2022 •
edited
Loading

Checklist

  • This is not a new feature or an enhancement to the Filecoin protocol. If it is, please open an FIP issue.
  • This is not a new feature request. If it is, please file a feature request instead.
  • This is not brainstorming ideas. If you have an idea you'd like to discuss, please open a new discussion on the venus forum and select the category as Ideas.
  • I have a specific, actionable, and well motivated improvement to propose.

Venus component

  • venus daemon - [chain service] chain sync
  • venus auth - [chain service] authentication
  • venus messager - [chain service] message management (mpool)
  • venus gateway - [chain service] gateway
  • venus miner - [chain service] mining and block production
  • venus sealer/worker - sealing
  • venus sealer - proving (WindowPoSt)
  • venus market - storage deal
  • venus market - retrieval deal
  • venus market - data transfer
  • venus light-weight client
  • venus JSON-RPC API
  • Other

Improvement Suggestion

间接的引用，没有意义，无效代码。删除这几个重定向类型的代码。
@Fatman13 Fatman13 added V-venus-gateway venus gateway component CU-chain-service Venus chain service related issues labels May 30, 2022
@Fatman13 Fatman13 moved this to Todo in Venus Project Jun 10, 2022
@Fatman13 Fatman13 mentioned this issue Jun 10, 2022
Closed
39 tasks
@LinZexiao LinZexiao linked a pull request Jun 14, 2022 that will close this issue
fix: venus-venus: change to build docker image from local repo #4912
Merged
5 tasks
@LinZexiao LinZexiao removed a link to a pull request Jun 14, 2022
fix: venus-venus: change to build docker image from local repo #4912
Merged
5 tasks
@LinZexiao LinZexiao mentioned this issue Jun 14, 2022
Merged
5 tasks
@LinZexiao LinZexiao moved this to In Review in Venus Project Jun 14, 2022
@LinZexiao LinZexiao moved this from In Review to In Progress in Venus Project Jun 14, 2022
Repository owner moved this from In Progress to Done in Venus Project Jun 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hunjixin hunjixin

@LinZexiao LinZexiao

Labels
CU-chain-service Venus chain service related issues V-venus-gateway venus gateway component
Projects
Venus Project
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: venus#4895 gateway : gateway rm indirect api ipfs-force-community/sophon-gateway
3 participants
@Fatman13 @hunjixin @LinZexiao