diff --git a/app/assets/tailwind/application.css b/app/assets/tailwind/application.css index c55530db..a3a9bf10 100644 --- a/app/assets/tailwind/application.css +++ b/app/assets/tailwind/application.css @@ -387,3 +387,49 @@ nav.pagy a.current { .help-text { @apply mt-2 text-xs text-gray-500 m-0; } + +/* Beacon Configuration Admin View styles */ +.beacon-online { + display: inline-block; + width: 10px; + height: 10px; + background-color: #10b981; + border-radius: 50%; + animation: pulse-green 2s infinite; +} + +.beacon-online-large { + display: inline-block; + width: 16px; + height: 16px; + background-color: #10b981; + border-radius: 50%; + animation: pulse-green 2s infinite; +} + +.beacon-offline { + display: inline-block; + width: 10px; + height: 10px; + background-color: #ef4444; + border-radius: 50%; +} + +.beacon-offline-large { + display: inline-block; + width: 16px; + height: 16px; + background-color: #ef4444; + border-radius: 50%; +} + +@keyframes pulse-green { + 0%, 100% { + opacity: 1; + box-shadow: 0 0 0 0 rgba(16, 185, 129, 0.7); + } + 50% { + opacity: 0.8; + box-shadow: 0 0 0 6px rgba(16, 185, 129, 0); + } +} diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb index 953bb1f6..6997b648 100644 --- a/app/controllers/application_controller.rb +++ b/app/controllers/application_controller.rb @@ -20,4 +20,8 @@ def current_provider end end helper_method :current_provider + + def non_contributor_redirect_path + topics_path + end end diff --git a/app/controllers/beacons_controller.rb b/app/controllers/beacons_controller.rb new file mode 100644 index 00000000..1a58461c --- /dev/null +++ b/app/controllers/beacons_controller.rb @@ -0,0 +1,99 @@ +class BeaconsController < ApplicationController + include Authentication + + before_action :set_beacon, only: %i[show edit update regenerate_key revoke_key] + before_action :prepare_associations, only: %i[new edit] + + def index + @beacons = Beacon.order(created_at: :desc) + end + + def new + @beacon = Beacon.new + end + + def create + success, @beacon, api_key = Beacons::Creator.new.call(beacon_params) + + if success + flash[:notice] = "Beacon was successfully provisioned. API Key: #{api_key}" + redirect_to beacon_path(@beacon, api_key: api_key) + else + prepare_associations + render :new, status: :unprocessable_entity + end + end + + def show; end + + def edit; end + + def update + if @beacon.update(beacon_params) + redirect_to @beacon, notice: "Beacon was successfully updated." + else + prepare_associations + render :edit, status: :unprocessable_entity + end + end + + def regenerate_key + _, api_key = Beacons::KeyRegenerator.new.call(@beacon) + flash[:notice] = "API key has been successfully regenerated. API Key: #{api_key}" + redirect_to beacon_path(@beacon, api_key: api_key) + + rescue => StandardError + flash[:alert] = "API key could not be regenerated." + redirect_to @beacon + end + + def filter_options + topics = if params[:language_id].present? + Topic.active.where(language_id: params[:language_id]).order(:title) + else + Topic.active.order(:title) + end + + providers = if params[:region_id].present? + Provider.joins(:branches).where(branches: { region_id: params[:region_id] }).distinct.order(:name) + else + Provider.order(:name) + end + + render json: { + topics: topics.select(:id, :title), + providers: providers.select(:id, :name), + } + end + + def revoke_key + api_key = @beacon.revoke! + flash[:notice] = "API key has been successfully revoked." + redirect_to @beacon + + rescue => StandardError + flash[:alert] = "API key could not be revoked." + redirect_to @beacon + end + + def non_contributor_redirect_path + root_path + end + + private + + def set_beacon + @beacon = Beacon.find(params[:id]) + end + + def prepare_associations + @languages = Language.order(:name) + @providers = Provider.order(:name) + @regions = Region.order(:name) + @topics = Topic.active.order(:title) + end + + def beacon_params + params.require(:beacon).permit(:name, :language_id, :region_id, provider_ids: [], topic_ids: []) + end +end diff --git a/app/controllers/concerns/authentication.rb b/app/controllers/concerns/authentication.rb index ba581279..79ca3553 100644 --- a/app/controllers/concerns/authentication.rb +++ b/app/controllers/concerns/authentication.rb @@ -13,7 +13,7 @@ def allow_unauthenticated_access(**options) end def redirect_contributors - redirect_to topics_path unless Current.user.is_admin? + redirect_to non_contributor_redirect_path unless Current.user.is_admin? end private diff --git a/app/javascript/controllers/beacon_form_controller.js b/app/javascript/controllers/beacon_form_controller.js new file mode 100644 index 00000000..9dd0edb6 --- /dev/null +++ b/app/javascript/controllers/beacon_form_controller.js @@ -0,0 +1,51 @@ +import { Controller } from "@hotwired/stimulus" +import { get } from "@rails/request.js" + +export default class extends Controller { + static targets = ["languageSelect", "regionSelect", "topicsSelect", "providersSelect"] + static values = { filterUrl: String } + + connect() { + // Defer until select-tags children have initialized their TomSelect instances + requestAnimationFrame(() => this.fetchOptions()) + } + + fetchOptions() { + const params = new URLSearchParams() + const languageId = this.languageSelectTarget.value + const regionId = this.regionSelectTarget.value + + if (languageId) params.set("language_id", languageId) + if (regionId) params.set("region_id", regionId) + + get(`${this.filterUrlValue}?${params}`, { responseKind: "json" }) + .then(response => response.json) + .then(data => { + this.#updateSelect(this.topicsSelectTarget, data.topics, "title") + this.#updateSelect(this.providersSelectTarget, data.providers, "name") + }) + } + + #updateSelect(selectElement, items, textKey) { + const tomSelect = selectElement.tomselect + if (!tomSelect) return + + const previousValues = [].concat(tomSelect.getValue()) + + tomSelect.clear(true) + tomSelect.clearOptions() + + items.forEach(item => { + tomSelect.addOption({ value: String(item.id), text: item[textKey] }) + }) + + const stillValidValues = previousValues.filter(v => + items.some(item => String(item.id) === v) + ) + if (stillValidValues.length > 0) { + tomSelect.setValue(stillValidValues, true) + } + + tomSelect.refreshOptions(false) + } +} diff --git a/app/javascript/controllers/beacons_controller.js b/app/javascript/controllers/beacons_controller.js new file mode 100644 index 00000000..0cbd796e --- /dev/null +++ b/app/javascript/controllers/beacons_controller.js @@ -0,0 +1,24 @@ +import { Controller } from "@hotwired/stimulus" + +export default class extends Controller { + static targets = ["button", "beaconApiKey"] + + copyApiKey() { + const apiKey = this.beaconApiKeyTarget.textContent; + + navigator.clipboard.writeText(apiKey).then(() => { + const button = this.buttonTarget; + const originalText = button.textContent; + button.textContent = "Copied!"; + button.classList.add("text-green-600"); + + setTimeout(() => { + button.textContent = originalText; + button.classList.remove("text-green-600"); + }, 2000); + }).catch(err => { + console.error("Failed to copy:", err); + alert("Failed to copy API key to clipboard"); + }); + } +} diff --git a/app/models/beacon.rb b/app/models/beacon.rb index 40774a6b..015ce242 100644 --- a/app/models/beacon.rb +++ b/app/models/beacon.rb @@ -38,6 +38,9 @@ class Beacon < ApplicationRecord has_many :beacon_topics, dependent: :destroy has_many :topics, through: :beacon_topics + delegate :name, to: :region, prefix: true + delegate :name, to: :language, prefix: true + validates :name, presence: true validates :api_key_digest, presence: true, uniqueness: true validates :api_key_prefix, presence: true @@ -53,6 +56,20 @@ def revoked? revoked_at.present? end + def status_str + revoked? ? "Revoked" : "Active" + end + + # Get count of topics that match this beacon's configuration + def document_count + topics.count + end + + # Get count of actual document files attached to matching topics + def file_count + topics.joins(:documents_attachments).count + end + def accessible_blobs ActiveStorage::Blob .joins(:attachments) diff --git a/app/views/beacons/_form.html.erb b/app/views/beacons/_form.html.erb new file mode 100644 index 00000000..6f7361cc --- /dev/null +++ b/app/views/beacons/_form.html.erb @@ -0,0 +1,193 @@ +<%= form_with model: beacon, local: true, + data: { controller: "beacon-form", beacon_form_filter_url_value: filter_options_beacons_path } do |form| %> + <% if beacon.errors.any? %> +
+
+
+ + + +
+ +
+

+ <%= pluralize(beacon.errors.count, "error") %> prohibited this + beacon from being saved: +

+ +
+
    + <% beacon.errors.full_messages.each do |message| %> +
  • <%= message %>
    • + <% end %> +
+
+
+
+
+ <% end %> + +
+ +
+

+ Basic Information +

+
+ +
+ <%= form.label :name, class: "form-label" do %> + Beacon Name * + <% end %> + + <%= form.text_field :name, class: "form-input", placeholder: "e.g., Kampala-Central", required: true %> + +

+ A descriptive name for this beacon deployment +

+
+ + +
+

+ Document Access Filters +

+ +

+ Select which documents this beacon can access. Leave filters blank to + allow access to all documents. +

+
+ +
+ <%= form.label :language_id, "Language", class: "form-label" do %> + Language * + <% end %> + +
+ <%= form.select :language_id, + options_from_collection_for_select(@languages, :id, :name, beacon.language_id), + { include_blank: "Select a language" }, + class: "form-select", required: true, + data: { + action: "change->beacon-form#fetchOptions", + "beacon-form-target": "languageSelect" + } %> + + <%= render "shared/dropdown_arrow" %> +
+ +

+ Language for documents this beacon will access +

+
+ +
+ <%= form.label :region_id, "Region", class: "form-label" do %> + Region * + <% end %> + +
+ <%= form.select :region_id, + options_from_collection_for_select(@regions, :id, :name, beacon.region_id), + { include_blank: "Select a region" }, + class: "form-select", required: true, + data: { + action: "change->beacon-form#fetchOptions", + "beacon-form-target": "regionSelect" + } %> + + <%= render "shared/dropdown_arrow" %> +
+ +

Geographic region for this beacon

+
+ +
+ <%= form.label :provider_ids, "Providers", class: "form-label" %> + + <%= form.select :provider_ids, + options_from_collection_for_select(@providers, :id, :name, beacon.provider_ids), + { prompt: "Start typing to select providers", include_blank: true }, + { multiple: true, + data: { + "allow-new": "false", + "allow-clear": "true", + "select-tags-target": "tagList", + "dropdown-parent": "body", + "beacon-form-target": "providersSelect" + } + } %> + +

+ Select which content providers this beacon can access +

+
+ +
+ <%= form.label :topic_ids, "Topics", class: "form-label" %> + + <%= form.select :topic_ids, + options_from_collection_for_select(@topics, :id, :title, beacon.topic_ids), + { prompt: "Start typing to select topics", include_blank: true }, + { multiple: true, + data: { + "allow-new": "false", + "allow-clear": "true", + "select-tags-target": "tagList", + "dropdown-parent": "body", + "beacon-form-target": "topicsSelect" + } + } %> + +

+ Select which topics this beacon can access. Leave blank to allow access + to all topics. +

+
+
+ + +
+
+ <%= form.submit beacon.persisted? ? "Update Beacon" : "Provision Beacon", class: "bg-gradient-green text-white font-semibold py-3 px-8 rounded-lg border-0 cursor-pointer text-sm shadow-md transition-all hover:-translate-y-0.5" %> + + <%= link_to "Back to Beacons", beacons_path, + class: "bg-gray-100 text-gray-500 font-semibold py-3 px-8 rounded-lg no-underline text-sm transition-all hover:bg-gray-200" %> +
+ + <% if beacon.persisted? %> +
+ Last updated: <%= beacon.updated_at.strftime("%B %d, %Y at %I:%M %p") %> +
+ <% end %> +
+<% end %> diff --git a/app/views/beacons/edit.html.erb b/app/views/beacons/edit.html.erb new file mode 100644 index 00000000..afe41e03 --- /dev/null +++ b/app/views/beacons/edit.html.erb @@ -0,0 +1,41 @@ +<% content_for :title, "Edit Beacon" %> + +
+
+ + +
+
+ <%= link_to beacon_path(@beacon), class: "text-gray-500 hover:text-gray-700 transition-colors" do %> + + + + <% end %> +

+ Edit Beacon +

+
+ + + +
+ + +
+ +
+ <%= render "form", beacon: @beacon %> +
+
+
+
diff --git a/app/views/beacons/index.html.erb b/app/views/beacons/index.html.erb new file mode 100644 index 00000000..a63ee2e0 --- /dev/null +++ b/app/views/beacons/index.html.erb @@ -0,0 +1,171 @@ +<% content_for :title, "Beacon Management" %> + +
+
+ + +
+
+
+
+
+ + + +
+

Beacon Management

+
+

Monitor system beacons deployed in the field

+
+ +
+
+
+ Active: <%= @beacons.count { |b| !b.revoked? } %> / + Revoked: <%= @beacons.count { |b| b.revoked? } %> +
+
+ + <%= link_to new_beacon_path, class: "bg-gradient-green text-white py-3.5 px-7 rounded-xl no-underline font-semibold text-sm inline-flex items-center gap-2 shadow-green transition-all hover:-translate-y-0.5", data: {turbo: false } do %> + + + + Provision New Beacon + <% end %> +
+
+
+ + +
+
+ + + + + + + + + + + + + + <% if @beacons.any? %> + <% @beacons.each do |beacon| %> + + + + + + + + + + <% end %> + <% else %> + + + + <% end %> + +
StatusBeacon NameCurrent Manifest VersionLatest Manifest VersionUp to Date?Last Seen atLast Sync at
+
+ + + <%= beacon.revoked? ? 'Revoked' : 'Active' %> + +
+ 		+ <%= link_to beacon_path(beacon), class: "no-underline" do %> +
+
+ + + +
+
+
<%= beacon.name %>
+
<%= beacon.api_key_prefix %>***
+
+
+ <% end %> + 		+ + current manifest + + + + latest manifest + + + + is up to date? + + + + last seen at + + + + last sync at + +
+
+ + + +

No beacons provisioned yet

+

Get started by provisioning your first beacon.

+
+
+
+
+ + +
+
+
+
+

Total Beacons

+

<%= @beacons.count %>

+
+
+ + + +
+
+
+ +
+
+
+

Online Now

+

<%= @beacons.count { |b| b[:online] } %>

+
+
+ + + +
+
+
+ +
+
+
+

Offline

+

<%= @beacons.count { |b| !b[:online] } %>

+
+
+ + + +
+
+
+
+
+
diff --git a/app/views/beacons/new.html.erb b/app/views/beacons/new.html.erb new file mode 100644 index 00000000..c9cca61a --- /dev/null +++ b/app/views/beacons/new.html.erb @@ -0,0 +1,60 @@ +<% content_for :title, "Provision New Beacon" %> + +
+
+ + +
+
+ <%= link_to beacons_path, class: "text-gray-500 hover:text-gray-700 transition-colors" do %> + + + + <% end %> +

+ Provision New Beacon +

+
+ + + +
+ + +
+ +
+ <%= render "form", beacon: @beacon %> +
+
+ + +
+
+
+ + + +
+
+

How Document Filtering Works

+
    +
  • Required fields: Language and Region must be selected for each beacon
    • +
  • Providers: Select specific content providers this beacon can access, or leave blank for all providers
    • +
  • Topics: Select specific topics this beacon can access, or leave blank for all topics
    • +
  • API Key generation: A secure API key will be automatically generated upon provisioning
    • +
  • Security: Keep the API key secure - it will only be shown once during creation
    • +
+
+
+
+
+
diff --git a/app/views/beacons/show.html.erb b/app/views/beacons/show.html.erb new file mode 100644 index 00000000..b79cdba1 --- /dev/null +++ b/app/views/beacons/show.html.erb @@ -0,0 +1,285 @@ +<% content_for :title, @beacon.name %> + +
+
+ + +
+
+
+
+
+ + + +
+

<%= @beacon.name %>

+
+
+ +
+ <%= link_to edit_beacon_path(@beacon), class: "bg-gradient-blue text-white py-3.5 px-7 rounded-xl no-underline font-semibold text-sm inline-flex items-center gap-2 shadow-blue transition-all hover:-translate-y-0.5", data: {turbo: false } do %> + + + + Edit Beacon + <% end %> +
+
+ + + +
+ + +
+
+
+
+

Status

+

+ <%= @beacon.status_str %> +

+
+
+ + <% if @beacon.revoked? %> + + <% else %> + + <% end %> + +
+
+
+ +
+
+
+

Files Included on Beacon

+

<%= @beacon.document_count %>

+
+
+ + + +
+
+
+ +
+
+
+

Topics Included

+

<%= @beacon.file_count %>

+
+
+ + + +
+
+
+
+ + +
+ + +
+
+

+ + + + Beacon Details +

+
+
+
+
+
ID
+
<%= @beacon.id %>
+
+
+
API Key Prefix
+
<%= @beacon.api_key_prefix %>***
+
+
+
Created
+
<%= @beacon.created_at.strftime("%B %d, %Y at %I:%M %p") %>
+
+
+
Updated
+
<%= @beacon.updated_at.strftime("%B %d, %Y at %I:%M %p") %>
+
+
+
+
+ + +
+
+

+ + + + API Key +

+
+
+ <% if params[:api_key] %> +
+
+ ✓ Beacon Provisioned - API Key Generated + +
+ <%= params[:api_key] %> +
+
+
+
+ + + +
+
+

Save This Key

+

+ Copy this API key and configure it in your beacon deployment. This page will only show the full key immediately after creation. You can regenerate a new key anytime if needed. +

+
+
+
+ <% else %> +
+
+ API Key Prefix +
+ <%= @beacon.api_key_prefix %>*** +
+
+
+
+ + + +
+
+

API Key Active

+

+ The API key for this beacon is active (shown by prefix above). The full key was displayed when the beacon was first created. If you need a new key, use the regenerate button below. +

+
+
+
+
+ <%= button_to regenerate_key_beacon_path(@beacon), method: :post, + class: "w-full bg-amber-600 hover:bg-amber-700 text-white font-semibold py-3 px-4 rounded-lg transition-colors inline-flex items-center justify-center gap-2", + data: { turbo_confirm: "Are you sure? This will invalidate the current API key and generate a new one. Any beacons using the old key will stop working." } do %> + + + + Regenerate API Key + <% end %> + + <%= button_to revoke_key_beacon_path(@beacon), method: :post, + class: "w-full bg-red-600 hover:bg-red-700 text-white font-semibold py-3 px-4 rounded-lg transition-colors inline-flex items-center justify-center gap-2", + data: { turbo_confirm: "Are you sure? This will invalidate the current API key. Any beacons using the old key will stop working." } do %> + + + + Revoke API Key + <% end %> +
+ <% end %> +
+
+
+ + +
+
+

+ + + + Document Access Configuration +

+
+
+
+
+

Basic Configuration

+
+
+
Language
+
+ + <%= @beacon.language_name %> + +
+
+
+
Region
+
+ + <%= @beacon.region_name %> + +
+
+
+
+
+

Access Filters

+
+
+
Providers
+
+ <% if @beacon.providers.any? %> +
+ <% @beacon.providers.each do |provider| %> + + <%= provider.name %> + + <% end %> +
+ <% else %> + All Providers + <% end %> +
+
+
+
Topics
+
+ <% if @beacon.topics.any? %> +
+ <% @beacon.topics.each do |topic| %> + + <%= topic.title %> + + <% end %> +
+ <% else %> + All Topics + <% end %> +
+
+
+
+
+
+
+ +
+
diff --git a/app/views/layouts/_sidebar.html.erb b/app/views/layouts/_sidebar.html.erb index c65b9a46..aecd77a1 100644 --- a/app/views/layouts/_sidebar.html.erb +++ b/app/views/layouts/_sidebar.html.erb @@ -59,6 +59,13 @@ <% end %> +
  • + <%= link_to beacons_path, class: "sidebar__link #{active_link_class(beacons_path)}" do %> + 📡 + Beacon Management + <% end %> +
    • +
  • <%= link_to import_reports_path, class: "sidebar__link #{active_link_class(import_reports_path)}" do %> 📊 diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb index 696d18b7..2ab66c1c 100644 --- a/app/views/layouts/application.html.erb +++ b/app/views/layouts/application.html.erb @@ -45,12 +45,7 @@
    - Built with - - - - by - Ruby for Good +

    Built with ❤️ by Ruby for Good

    diff --git a/bin/docker-entrypoint.dev b/bin/docker-entrypoint.dev index 6022d807..10104335 100755 --- a/bin/docker-entrypoint.dev +++ b/bin/docker-entrypoint.dev @@ -9,5 +9,8 @@ bundle check || bundle install # If running the rails server then create or migrate existing database ./bin/rails db:prepare +# Remove compiled asset manifest so Propshaft serves assets dynamically in development +rm -f /app/public/assets/.manifest.json + # Execute the main command passed to docker run exec "$@" diff --git a/config/routes.rb b/config/routes.rb index 41fda43a..d044bbbf 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -21,6 +21,16 @@ put :provider, on: :collection end + resources :beacons, except: :destroy do + collection do + get :filter_options + end + member do + post :regenerate_key + post :revoke_key + end + end + # Render dynamic PWA files from app/views/pwa/* (remember to link manifest in application.html.erb) # get "manifest" => "rails/pwa#manifest", as: :pwa_manifest # get "service-worker" => "rails/pwa#service_worker", as: :pwa_service_worker diff --git a/db/seeds.rb b/db/seeds.rb index a89cf071..6b9540bc 100644 --- a/db/seeds.rb +++ b/db/seeds.rb @@ -81,7 +81,7 @@ { email: "admin@mail.com", password: "test123", is_admin: true }, { email: "me@mail.com", password: "test123" }, ].map do |user_data| - user = User.find_or_initialize_by(email: user_data[:email]).tap do |u| + User.find_or_initialize_by(email: user_data[:email]).tap do |u| u.password = user_data[:password] u.is_admin = user_data[:is_admin] || false u.providers << Provider.first unless u.is_admin @@ -95,7 +95,7 @@ end 10.times do - user = User.find_or_initialize_by(email: Faker::Internet.email).tap do |u| + User.find_or_initialize_by(email: Faker::Internet.email).tap do |u| u.password = Faker::Internet.password u.is_admin = false u.providers << Provider.first unless u.is_admin diff --git a/docs/model_relationships.md b/docs/model_relationships.md new file mode 100644 index 00000000..0f86e475 --- /dev/null +++ b/docs/model_relationships.md @@ -0,0 +1,151 @@ +# Model Relationships + +_(A Draft of v3 Architecture, helpful for me to understand and also to onboard others)_ + +## The big picture + +SkillRx delivers continuing medical education (CME) to medical practitioners in remote areas. Understanding that mission makes the model hierarchy intuitive. + +**Topic is the protagonist.** A Topic is the actual CME content — the reason the app exists. Everything else exists to produce Topics, organize them, or get the right ones to the right place. + +**Provider is the author.** A Provider (an NGO, hospital system, or medical publisher) produces Topics in a given Language. Without Providers there is no content; without content there is no app. + +**Beacon is the delivery hero.** A Beacon represents a physical deployment — a device or installation at a remote clinic. It is configured for a specific Language and Region, and acts as a gatekeeper: filtering which Topics (and from which Providers) that location can access. + +The narrative arc of the data: + +> A **Provider** produces **Topics** (CME materials) in a given **Language**. A **Beacon** is deployed at a remote clinic — configured for a specific **Region** and **Language** — and serves as the gatekeeper that filters which **Topics** (and from which **Providers**) that clinic's device can access. + +--- + +## 1. Content layer + +_What the app exists to deliver._ + +### Topic + +The central domain object. Every other model either produces, organizes, or delivers Topics. + +- Belongs to one **Language** +- Belongs to one **Provider** +- Has many **Beacons** through BeaconTopics (which beacons serve this topic) +- Has many **Tags** via `acts_as_taggable_on` (for search and discoverability) + +### Provider + +The organization that authors content. + +- Has many **Topics** (a topic belongs to exactly one provider) +- Has many **Branches** → and through them, many **Regions** (a provider can operate in multiple regions) +- Has many **Contributors** → and through them, many **Users** (the people who manage that provider's content) +- Has many **Beacons** through BeaconProviders + +### Language + +The language a Topic is written in; also used to scope a Beacon's content. + +- Has many **Topics** +- Has many **Providers** through Topics (derived: which providers have content in this language) +- Has many **Beacons** (each beacon is assigned exactly one language) + +--- + +## 2. Delivery layer + +_How content reaches remote locations._ + +### Beacon + +The delivery object through which medical practitioners access topics. It's managed by **Provider(s)** to deliver **Topics** to end users. + +- Belongs to one **Language** (required — scopes topics to one language) +- Belongs to one **Region** (required — scopes providers to those with a branch in this region) +- Has many **Providers** through BeaconProviders (optional further restriction) +- Has many **Topics** through BeaconTopics (optional further restriction) + +### Region + +The geographic area a Beacon serves; also how Providers are scoped to locations. + +- Has many **Branches** (a branch is a provider's presence in a region) +- Has many **Providers** through Branches +- Has many **Beacons** (each beacon is assigned exactly one region) + +### Branch + +Links a Provider to a Region. The join that makes geographic filtering possible. + +- Belongs to one **Provider** +- Belongs to one **Region** + +--- + +## 3. People layer + +_Who manages the content._ + +### User + +An administrator of the system. + +- Has many **Contributors** → and through them, many **Providers** +- Non-admin users must be affiliated with at least one Provider (enforced by model validation) +- Admins have no provider requirement + +### Contributor + +Records a User's affiliation with a Provider. + +- Belongs to one **User** +- Belongs to one **Provider** + +--- + +## 4. Taxonomy layer + +_How content is discovered and grouped._ + +### Tag + +A keyword applied to Topics to support search and filtering. + +- Applied to Topics via `acts_as_taggable_on` +- Can be linked to other Tags as **cognates** (semantically equivalent terms — e.g. synonyms across dialects or languages) + +### TagCognate + +A self-referential join between two Tags that are considered cognates. Relationships are deduplicated (no reverse duplicates allowed). When a tag is added to a topic, its cognates are automatically added as well. + +- Belongs to one **Tag** (as `tag`) +- Belongs to one **Tag** (as `cognate`) + +--- + +## Join / pivot model summary + +| Model | Joins | Layer | +| ------------------ | ---------------------------- | -------- | +| **Branch** | Provider ↔ Region | Delivery | +| **Contributor** | User ↔ Provider | People | +| **BeaconProvider** | Beacon ↔ Provider | Delivery | +| **BeaconTopic** | Beacon ↔ Topic | Delivery | +| **TagCognate** | Tag ↔ Tag (self-referential) | Taxonomy | + +--- + +## Diagram + +``` +[People] +User ──── Contributor ──────────────────────────┐ + ↓ +[Content] Provider ──── Branch ──── Region +Language ──── Topic ───────────────────────↑ ↑ + │ │ + [Tag] ── TagCognate │ + │ +[Delivery] Beacon ──────────────────────┘ + (belongs_to Language, Region) + (has_many Providers via BeaconProvider) + (has_many Topics via BeaconTopic) +``` diff --git a/spec/requests/beacons_spec.rb b/spec/requests/beacons_spec.rb new file mode 100644 index 00000000..656c79f2 --- /dev/null +++ b/spec/requests/beacons_spec.rb @@ -0,0 +1,176 @@ +require "rails_helper" + +RSpec.describe "/beacons", type: :request do + let(:user) { create(:user, :admin) } + let(:region) { create(:region) } + let(:language) { create(:language) } + let(:valid_attributes) do + { name: "New Beacon", + language_id: language.id, + region_id: region.id, + } + end + let(:invalid_attributes) { { name: "" } } + + before do + sign_in(user) + end + + describe "GET /index" do + it "renders a successful response" do + create(:beacon) + + get beacons_url + + expect(response).to be_successful + end + end + + describe "GET /show" do + it "renders a successful response" do + beacon = create(:beacon) + + get beacon_url(beacon) + + expect(response).to be_successful + end + end + + describe "GET /new" do + it "renders a successful response" do + get new_beacon_url + + expect(response).to be_successful + end + end + + describe "GET /edit" do + let(:beacon) { create(:beacon) } + + it "renders a successful response" do + get edit_beacon_url(beacon) + + expect(response).to be_successful + end + end + + describe "POST /create" do + context "with valid parameters" do + it "creates a new beacon" do + expect { + post beacons_url, params: { beacon: valid_attributes } + }.to change(Beacon, :count).by(1) + end + + it "redirects to the created beacon" do + post beacons_url, params: { beacon: valid_attributes } + + expect(response).to have_http_status(302) + expect(assigns(:beacon)).to eq(Beacon.last) + end + end + + context "with invalid parameters" do + it "does not create a new beacon" do + expect { + post beacons_url, params: { beacon: invalid_attributes } + }.to change(Beacon, :count).by(0) + end + + it "renders a response with 422 status (i.e. to display the 'new' template)" do + post beacons_url, params: { beacon: invalid_attributes } + + expect(response).to have_http_status(:unprocessable_content) + end + end + end + + describe "PATCH /update" do + context "with valid parameters" do + let(:updated_region) { create(:region) } + let(:updated_language) { create(:language) } + let(:new_attributes) do + { name: "Updated Beacon", + language_id: updated_language.id, + region_id: updated_region.id, + } + end + + it "updates the requested beacon" do + beacon = create(:beacon) + + patch beacon_url(beacon), params: { beacon: new_attributes } + beacon.reload + + expect(beacon.name).to eq("Updated Beacon") + expect(beacon.language).to eq(updated_language) + expect(beacon.region).to eq(updated_region) + end + + it "redirects to the beacon" do + beacon = create(:beacon) + + patch beacon_url(beacon), params: { beacon: new_attributes } + beacon.reload + + expect(response).to redirect_to(beacon_url(beacon)) + end + end + + context "with invalid parameters" do + it "renders a response with 422 status (i.e. to display the 'edit' template)" do + beacon = create(:beacon) + + patch beacon_url(beacon), params: { beacon: invalid_attributes } + + expect(response).to have_http_status(:unprocessable_content) + end + end + end + + describe "POST /regenerate_key" do + let(:beacon) { create(:beacon) } + subject { post regenerate_key_beacon_url(beacon) } + + it "redirects to the beacon" do + subject + + expect(response).to have_http_status(302) + expect(assigns(:beacon)).to eq(beacon) + end + + context "when there is an error" do + before { allow(Beacons::KeyRegenerator.new).to receive(:call).and_raise("Error") } + + it "redirects to the beacon" do + subject + + expect(response).to have_http_status(302) + expect(assigns(:beacon)).to eq(beacon) + end + end + end + + describe "POST /revoke_key" do + include ActiveSupport::Testing::TimeHelpers + + let(:beacon) { create(:beacon) } + subject { post revoke_key_beacon_url(beacon) } + + it "redirects to the beacon" do + subject + + expect(response).to redirect_to(beacon_url(beacon)) + end + + context "when there is an error" do + before { allow(beacon).to receive(:revoke!).and_raise("Error") } + + it "redirects to the beacon" do + subject + + expect(response).to redirect_to(beacon_url(beacon)) + end + end + end +end