Description:
merge
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Author
-
r840:b948bb241476 - - 9 files changed: 113 inserted, 49 deleted
| @@ -0,0 +1,25 | |||
|
|
1 | + = simple_form_for(@announcement) do |f| | |
|
|
2 | + - if @announcement.errors.any? | |
|
|
3 | + .form-row | |
|
|
4 | + .col-sm-12 | |
|
|
5 | + #error_explanation.alert.alert-dismissable.alert-danger | |
|
|
6 | + %button.close{type: "button", data: { dismiss: "alert"}, aria: {hidden: "true"}} × | |
|
|
7 | + %h3= "เกิดปัญหาในการบันทึกข้อมูลเนื่องจาก" | |
|
|
8 | + %ul | |
|
|
9 | + - @announcement.errors.full_messages.each do |message| | |
|
|
10 | + %li= message | |
|
|
11 | + .form-row | |
|
|
12 | + .col-md-6.col-12 | |
|
|
13 | + -# = f.input :lock_version, as: :hidden | |
|
|
14 | + = f.input :title | |
|
|
15 | + = f.input :notes, label: 'Notes (shown internally, used to organize announcements)' | |
|
|
16 | + = f.input :body | |
|
|
17 | + = f.input :author | |
|
|
18 | + = f.input :published | |
|
|
19 | + = f.input :frontpage, label: 'Display in the front page only?' | |
|
|
20 | + = f.input :on_nav_bar, label: 'Show on menu bar?', wrapper: :custom_boolean | |
|
|
21 | + = f.input :contest_only, label: 'Display in contest only?' | |
|
|
22 | + = f.submit "Create", class: 'btn btn-primary' | |
|
|
23 | + -if content_for?(:form_buttons) | |
|
|
24 | + = yield(:form_buttons) | |
|
|
25 | + / = link_to 'Back', announcements_path, class: 'btn btn-default' | |
| @@ -0,0 +1,5 | |||
|
|
1 | + class AddOnNavBarToAnnouncement < ActiveRecord::Migration[5.2] | |
|
|
2 | + def change | |
|
|
3 | + add_column :announcements, :on_nav_bar, :boolean, default: false | |
|
|
4 | + end | |
|
|
5 | + end | |
| @@ -108,9 +108,9 | |||
|
|
108 | 108 | end |
|
|
109 | 109 | end |
|
|
110 | 110 | |
|
|
111 | 111 | private |
|
|
112 | 112 | |
|
|
113 | 113 | def announcement_params |
|
|
114 | - params.require(:announcement).permit(:author, :body, :published, :frontpage, :contest_only, :title) | |
|
|
114 | + params.require(:announcement).permit(:author, :body, :published, :frontpage, :contest_only, :title, :on_nav_bar) | |
|
|
115 | 115 | end |
|
|
116 | 116 | end |
| @@ -1,12 +1,13 | |||
|
|
1 | 1 | require 'ipaddr' |
|
|
2 | 2 | |
|
|
3 | 3 | class ApplicationController < ActionController::Base |
|
|
4 | 4 | protect_from_forgery |
|
|
5 | 5 | |
|
|
6 | 6 | before_action :current_user |
|
|
7 | + before_action :nav_announcement | |
|
|
7 | 8 | |
|
|
8 | 9 | SINGLE_USER_MODE_CONF_KEY = 'system.single_user_mode' |
|
|
9 | 10 | MULTIPLE_IP_LOGIN_CONF_KEY = 'right.multiple_ip_login' |
|
|
10 | 11 | WHITELIST_IGNORE_CONF_KEY = 'right.whitelist_ignore' |
|
|
11 | 12 | WHITELIST_IP_CONF_KEY = 'right.whitelist_ip' |
|
|
12 | 13 | |
| @@ -19,12 +20,16 | |||
|
|
19 | 20 | # Returns the current logged-in user (if any). |
|
|
20 | 21 | def current_user |
|
|
21 | 22 | return nil unless session[:user_id] |
|
|
22 | 23 | @current_user ||= User.find(session[:user_id]) |
|
|
23 | 24 | end |
|
|
24 | 25 | |
|
|
26 | + def nav_announcement | |
|
|
27 | + @nav_announcement = Announcement.where(on_nav_bar: true) | |
|
|
28 | + end | |
|
|
29 | + | |
|
|
25 | 30 | def admin_authorization |
|
|
26 | 31 | return false unless check_valid_login |
|
|
27 | 32 | user = User.includes(:roles).find(session[:user_id]) |
|
|
28 | 33 | unless user.admin? |
|
|
29 | 34 | unauthorized_redirect |
|
|
30 | 35 | return false |
| @@ -1,34 +1,45 | |||
|
|
1 | - .container-fluid | |
|
|
2 | - %h1 Editing announcement | |
|
|
3 | - = error_messages_for :announcement | |
|
|
4 | - .row | |
|
|
5 | - .col-md-6 | |
|
|
6 | - = form_for(@announcement) do |f| | |
|
|
7 | - .form-group | |
|
|
8 | - %label Title | |
|
|
9 | - = f.text_field :title, class: 'form-control' | |
|
|
10 | - .form-group | |
|
|
11 | - %label Notes | |
|
|
12 | - (shown internally, used to organize announcements) | |
|
|
13 | - = f.text_field :notes, class: 'form-control' | |
|
|
14 | - .form-group | |
|
|
15 |
- %label |
|
|
|
16 |
- = f.text_ |
|
|
|
17 | - .form-group | |
|
|
18 |
- %label |
|
|
|
19 | - = f.text_field :author, class: 'form-control' | |
|
|
20 | - .checkbox | |
|
|
21 | - %label | |
|
|
22 | - = f.check_box :published | |
|
|
23 | - Published | |
|
|
24 | - .checkbox | |
|
|
25 | - %label | |
|
|
26 | - = f.check_box :frontpage | |
|
|
27 | - Show on front page? | |
|
|
28 | - .checkbox | |
|
|
29 | - %label | |
|
|
30 | - = f.check_box :contest_only | |
|
|
31 | - Show only in contest? | |
|
|
32 | - = f.submit "Update", class: 'btn btn-primary' | |
|
|
33 | - = link_to 'Show', @announcement, class: 'btn btn-default' | |
|
|
34 | - = link_to 'Back', announcements_path, class: 'btn btn-default' | |
|
|
1 | + %h1 Edit Announcement | |
|
|
2 | + | |
|
|
3 | + -content_for(:form_buttons) do | |
|
|
4 | + = link_to t(:back), announcements_path, class: 'card-link btn btn-secondary' | |
|
|
5 | + | |
|
|
6 | + = render 'form' | |
|
|
7 | + -# old style | |
|
|
8 | + .container-fluid | |
|
|
9 | + %h1 Editing announcement | |
|
|
10 | + = error_messages_for :announcement | |
|
|
11 | + .row | |
|
|
12 | + .col-md-6 | |
|
|
13 | + = form_for(@announcement) do |f| | |
|
|
14 | + .form-group | |
|
|
15 | + %label Title | |
|
|
16 | + = f.text_field :title, class: 'form-control' | |
|
|
17 | + .form-group | |
|
|
18 | + %label Notes | |
|
|
19 | + (shown internally, used to organize announcements) | |
|
|
20 | + = f.text_field :notes, class: 'form-control' | |
|
|
21 | + .form-group | |
|
|
22 | + %label Body | |
|
|
23 | + = f.text_area :body, class: 'form-control', style: 'height: 200px;' | |
|
|
24 | + .form-group | |
|
|
25 | + %label Author | |
|
|
26 | + = f.text_field :author, class: 'form-control' | |
|
|
27 | + .checkbox | |
|
|
28 | + %label | |
|
|
29 | + = f.check_box :published | |
|
|
30 | + Published | |
|
|
31 | + .checkbox | |
|
|
32 | + %label | |
|
|
33 | + = f.check_box :frontpage | |
|
|
34 | + Show on front page? | |
|
|
35 | + .checkbox | |
|
|
36 | + %label | |
|
|
37 | + = f.check_box :on_nav_bar | |
|
|
38 | + Show on top menu bar? | |
|
|
39 | + .checkbox | |
|
|
40 | + %label | |
|
|
41 | + = f.check_box :contest_only | |
|
|
42 | + Show only in contest? | |
|
|
43 | + = f.submit "Update", class: 'btn btn-primary' | |
|
|
44 | + = link_to 'Show', @announcement, class: 'btn btn-default' | |
|
|
45 | + = link_to 'Back', announcements_path, class: 'btn btn-default' | |
| @@ -1,14 +1,22 | |||
|
|
1 | 1 | %h1 New announcement |
|
|
2 | - = error_messages_for :announcement | |
|
|
3 | - = simple_form_for(@announcement) do |f| | |
|
|
4 | - .row | |
|
|
5 | - .col-md-6 | |
|
|
6 | - = f.input :title | |
|
|
7 | - = f.input :notes, label: 'Notes (shown internally, used to organize announcements)' | |
|
|
8 | - = f.input :body | |
|
|
9 | - = f.input :author | |
|
|
10 | - = f.input :published | |
|
|
11 | - = f.input :frontpage, label: 'Display in the front page only?' | |
|
|
12 | - = f.input :contest_only, label: 'Display in contest only?' | |
|
|
13 | - = f.button :submit, "Create", class: 'btn btn-primary' | |
|
|
14 | - = link_to 'Back', announcements_path, class: 'btn btn-default' | |
|
|
2 | + | |
|
|
3 | + -content_for(:form_buttons) do | |
|
|
4 | + = link_to t(:back), announcements_path, class: 'card-link btn btn-secondary' | |
|
|
5 | + | |
|
|
6 | + = render 'form' | |
|
|
7 | + | |
|
|
8 | + -# old style | |
|
|
9 | + = error_messages_for :announcement | |
|
|
10 | + = simple_form_for(@announcement) do |f| | |
|
|
11 | + .row | |
|
|
12 | + .col-md-6 | |
|
|
13 | + = f.input :title | |
|
|
14 | + = f.input :notes, label: 'Notes (shown internally, used to organize announcements)' | |
|
|
15 | + = f.input :body | |
|
|
16 | + = f.input :author | |
|
|
17 | + = f.input :published | |
|
|
18 | + = f.input :frontpage, label: 'Display in the front page only?' | |
|
|
19 | + = f.input :contest_only, label: 'Display in contest only?' | |
|
|
20 | + = f.button :submit, "Create", class: 'btn btn-primary' | |
|
|
21 | + = link_to 'Back', announcements_path, class: 'btn btn-default' | |
|
|
22 | + | |
| @@ -14,11 +14,14 | |||
|
|
14 | 14 | %b Published: |
|
|
15 | 15 | = h @announcement.published |
|
|
16 | 16 | %p |
|
|
17 | 17 | %b Show on front page: |
|
|
18 | 18 | = h @announcement.frontpage |
|
|
19 | 19 | %p |
|
|
20 | + %b Show on top menu bar: | |
|
|
21 | + = h @announcement.on_nav_bar | |
|
|
22 | + %p | |
|
|
20 | 23 | %b Show only in contest: |
|
|
21 | 24 | = h @announcement.contest_only |
|
|
22 | 25 | = link_to 'Edit', edit_announcement_path(@announcement) |
|
|
23 | 26 | | |
|
|
24 | 27 | = link_to 'Back', announcements_path |
| @@ -67,12 +67,18 | |||
|
|
67 | 67 | = add_menu( 'Submission Report', 'report', 'submission') |
|
|
68 | 68 | = add_menu( 'Login Report', 'report', 'login') |
|
|
69 | 69 | - if (ungraded = Submission.where('graded_at is null').where('submitted_at < ?', 1.minutes.ago).count) > 0 |
|
|
70 | 70 | =link_to "#{ungraded} backlogs!", |
|
|
71 | 71 | graders_list_path, |
|
|
72 | 72 | class: 'navbar-btn btn btn-default btn-warning', data: {toggle: 'tooltip'},title: 'Number of ungraded submission' |
|
|
73 | + / announcement | |
|
|
74 | + - @nav_announcement.each do |ann| | |
|
|
75 | + %p.navbar-text | |
|
|
76 | + = ann.body.html_safe | |
|
|
77 | + | |
|
|
78 | + | |
|
|
73 | 79 | |
|
|
74 | 80 | %ul.nav.navbar-nav.navbar-right |
|
|
75 | 81 | = add_menu("#{content_tag(:span,'',class: 'glyphicon glyphicon-question-sign')}".html_safe, 'main', 'help') |
|
|
76 | 82 | = add_menu("#{content_tag(:span,'',class: 'glyphicon glyphicon-comment')}".html_safe, 'messages', 'index', {title: I18n.t('menu.messages'), data: {toggle: 'tooltip'}}) |
|
|
77 | 83 | - if GraderConfiguration['system.user_setting_enabled'] |
|
|
78 | 84 | = add_menu("#{content_tag(:span,'',class: 'glyphicon glyphicon-cog', id: 'user_profile')}".html_safe, 'users', 'profile', {title: I18n.t('menu.settings'), data: {toggle: 'tooltip'}}) |
| @@ -7,24 +7,25 | |||
|
|
7 | 7 | # system, you should be using db:schema:load, not running all the migrations |
|
|
8 | 8 | # from scratch. The latter is a flawed and unsustainable approach (the more migrations |
|
|
9 | 9 | # you'll amass, the slower it'll run and the greater likelihood for issues). |
|
|
10 | 10 | # |
|
|
11 | 11 | # It's strongly recommended that you check this file into your version control system. |
|
|
12 | 12 | |
|
|
13 |
- ActiveRecord::Schema.define(version: 202 |
|
|
|
13 | + ActiveRecord::Schema.define(version: 2021_01_24_101028) do | |
|
|
14 | 14 | |
|
|
15 | 15 | create_table "announcements", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci", force: :cascade do |t| |
|
|
16 | 16 | t.string "author" |
|
|
17 | 17 | t.text "body" |
|
|
18 | 18 | t.boolean "published" |
|
|
19 | 19 | t.datetime "created_at", null: false |
|
|
20 | 20 | t.datetime "updated_at", null: false |
|
|
21 | 21 | t.boolean "frontpage", default: false |
|
|
22 | 22 | t.boolean "contest_only", default: false |
|
|
23 | 23 | t.string "title" |
|
|
24 | 24 | t.string "notes" |
|
|
25 | + t.boolean "on_nav_bar", default: false | |
|
|
25 | 26 | end |
|
|
26 | 27 | |
|
|
27 | 28 | create_table "contests", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci", force: :cascade do |t| |
|
|
28 | 29 | t.string "title" |
|
|
29 | 30 | t.boolean "enabled" |
|
|
30 | 31 | t.datetime "created_at", null: false |
You need to be logged in to leave comments.
Login now