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

#247 新規投稿時の公開範囲と共同編集者のデフォルト値の挙動を変更 #249

Merged
merged 1 commit into from
Jan 19, 2016

Conversation

nagodon
Copy link
Contributor

@nagodon nagodon commented Jan 18, 2016

#247 の対応になります。

@@ -40,11 +40,11 @@ var _SET_IMAGE_LABEL= '<%= jspUtil.label("knowledge.edit.set.image.path") %>';
var _LABEL_UPDATE = '<%= jspUtil.label("label.update") %>';
var _UPDATE_TITLE = '<%= jspUtil.label("knowledge.edit.title") %>';

<c:forEach var="group" items="${groups}" varStatus="status">
selectedGroups.push({label: '<%= jspUtil.out("group.label") %>', value: '<%= jspUtil.out("group.value") %>'});
<c:forEach var="selectedGroup" items="${groups}" varStatus="status">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

変数名の衝突が起きて正常動作しなかったため名前を変更しました。

クエリにグループが指定されてる場合はデフォルトで
公開範囲と共同編集者として選択済みにするように変更
@koda-masaru
Copy link
Contributor

PRありがとうございます。
一覧をグループで絞り込んで表示した場合、作成をすると、そのグループへの公開範囲がセットされることを確認しました。
想定通りの動作で問題無いかと思いますので、マージいたします。

koda-masaru added a commit that referenced this pull request Jan 19, 2016
#247 新規投稿時の公開範囲と共同編集者のデフォルト値の挙動を変更
@koda-masaru koda-masaru merged commit e0f0a19 into develop Jan 19, 2016
@koda-masaru koda-masaru deleted the fix/issue-247 branch January 19, 2016 13:32
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