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

Fix for https://github.com/ImbueNetwork/imbue/pull/269 #273

Merged
merged 2 commits into from
Nov 22, 2023
Merged

Conversation

f-gate
Copy link
Member

@f-gate f-gate commented Nov 21, 2023

  • Removed pallet-crowdfunding from workspace, (i simply dont have time to maintain a pallet we are not using).
  • Added a trivial test to ensure that the new code is working
  • Moved around the events to show the correct order of happening
  • Put the logic into the function where autofinalisation happens.

@f-gate f-gate changed the title Fix for https://github.com/ImbueNetwork/imbue/commit/445f8f6193afc78cec17e1d83f2ca53663d93822 Fix for https://github.com/ImbueNetwork/imbue/pull/269 Nov 21, 2023
@samelamin samelamin merged commit 691fff9 into main Nov 22, 2023
1 of 2 checks passed
@samelamin samelamin deleted the hotfix-1 branch November 22, 2023 17:10
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