🌐 AI搜索 & 代理 主页
Skip to content

Conversation

@berinhard
Copy link
Contributor

Related to #1666

@ewdurbin this is still a WIP and the current DB modeling doesn't reflect the scenario we've talked about recently. Anyway, comments, suggestions and feedbacks are welcome =)

@berinhard berinhard closed this Oct 30, 2020
@berinhard berinhard reopened this Oct 30, 2020
@berinhard berinhard marked this pull request as draft October 30, 2020 14:36
@berinhard berinhard requested a review from ewdurbin October 30, 2020 14:36
Copy link
Member

@ewdurbin ewdurbin left a comment

Choose a reason for hiding this comment

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

@berinhard in reviewing this, I'm no seeing anything that needs addressed outside of what's already been discussed ahead of the public facing parts launching.

We'll need a more in-depth cycle when implementing the workflow parts for sure though.

@ewdurbin ewdurbin changed the title Backend workflow for sponsorship applications User workflow for sponsorship applications Nov 10, 2020
@berinhard berinhard marked this pull request as ready for review November 10, 2020 22:10
@berinhard
Copy link
Contributor Author

@berinhard in reviewing this, I'm no seeing anything that needs addressed outside of what's already been discussed ahead of the public facing parts launching.

We'll need a more in-depth cycle when implementing the workflow parts for sure though.

Great @ewdurbin! I added a few more commits to fix the bugs that pop up during our demo and now this PR is ready for a final review + merge & deploy.

@berinhard
Copy link
Contributor Author

@ewdurbin I've already added the status field + date fields to Sponsorship in 6ff95a2

@ewdurbin ewdurbin merged commit e4a3d88 into master Nov 11, 2020
@ewdurbin ewdurbin deleted the feature/sponsorship-application-backend branch November 11, 2020 22:04
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