Skip to content

fix update_channel_as_owner to use new_channel_name#118

Closed
mnaamani wants to merge 1 commit intoJoystream:developmentfrom
mnaamani:wg-fix-update_channel_as_owner
Closed

fix update_channel_as_owner to use new_channel_name#118
mnaamani wants to merge 1 commit intoJoystream:developmentfrom
mnaamani:wg-fix-update_channel_as_owner

Conversation

@mnaamani
Copy link
Copy Markdown
Member

@mnaamani mnaamani commented Jan 9, 2020

No description provided.

@siman
Copy link
Copy Markdown
Contributor

siman commented Jan 15, 2020

Seems like this PR will conflict with other PR on this repo:
#124

And actually I also fixed this issue. @mnaamani you can see my fix on this line (also I renamed channel_name to handle):
https://github.com/Joystream/substrate-runtime-joystream/pull/124/files#diff-1fdbcd2edbf80fa54d527c6eb2a0c967R1168

To conclude: I would close this current PR.

@mnaamani mnaamani closed this Jan 16, 2020
@mnaamani mnaamani deleted the wg-fix-update_channel_as_owner branch May 22, 2020 20:34
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.

2 participants