Skip to content

Fix 巴哈 reply validation and update to meteor 1.12 #667

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

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

DW225
Copy link
Contributor

@DW225 DW225 commented Jun 4, 2022

As part of the upgrade the mongodb used might need some work.
Can be found in https://guide.meteor.com/1.7-migration.html

Copy link
Member

@SoftwareSing SoftwareSing left a comment

Choose a reason for hiding this comment

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

LGTM

升級蠻多東西的,大嘴更新前要記得備份一下,怕有問題

@@ -45,6 +45,7 @@ describe('method editUserAbout', function() {
});

it(`should success update user.about`, function() {
console.log('eeeeeeeeeeeeeeeeeeeeeeeeeeeee');
Copy link
Member

Choose a reason for hiding this comment

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

另外這行應該是不需要的吧

Copy link
Contributor Author

Choose a reason for hiding this comment

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

忘了刪www

Copy link
Contributor Author

Choose a reason for hiding this comment

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

刪掉了

@SoftwareSing
Copy link
Member

@mrbigmouth tag 一下,這個有空可以更新

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