-
-
Notifications
You must be signed in to change notification settings - Fork 44
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
Two bug-fixes #2405
Two bug-fixes #2405
Conversation
…sebin/task/#2396-non-monetary-bug
group-income Run #3364
Run Properties:
|
Project |
group-income
|
Branch Review |
sebin/task/#2396-non-monetary-bug
|
Run status |
Passed #3364
|
Run duration | 08m 50s |
Commit |
27736dbbdd ℹ️: Merge 4024c8552e8f1b5b460c6ac02a63ef72e6015657 into affa61592e03442878cea59bc60f...
|
Committer | Sebin Song |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
10
|
Skipped |
0
|
Passing |
111
|
View all changes introduced in this branch ↗︎ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx! Minor issue
@@ -6,7 +6,7 @@ modal-template(ref='modal' :a11yTitle='L("Add new members")') | |||
.c-container | |||
template(v-if='link') | |||
i18n.is-title-4(tag='h3') Share this link to grant access to your group. | |||
i18n.has-text-1(tag='p') After the onboarding period has ended, everyone will be asked to vote on whether or not a new member should be added. But for now, enjoy 60 free passes! | |||
i18n.has-text-1(tag='p') After the onboarding period has ended, everyone will be asked to vote on whether or not a new member should be added. But for now, enjoy 150 free passes! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you make this a variable so that we don't have to re-translate it next time it changes?
See issue for recommended variable value.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I jumped too early to making a fix and missed that part in the issue. updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
closes #2396
closes #2404