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

Spaces #1094

Draft
wants to merge 27 commits into
base: master
Choose a base branch
from
Draft

Spaces #1094

wants to merge 27 commits into from

Conversation

fbacall
Copy link
Member

@fbacall fbacall commented Apr 4, 2025

Summary of changes

  • Introduces the Space feature - independently managed spaces within TeSS, each with their own catalogue of events, materials etc. and some basic customization.
  • Spaces can be created by administrators, with a title, description, host, theme and logo.
  • Users can be given roles within a space (currently only role available is space administrator - who can change the customization of their space).
  • The current space is determined by looking at the Host header of the HTTP request. If no space exists for that host, the Default Space is set.
  • Any resources created in the current space are only visible in that space (unless a filter is enabled).
  • Currently Events, Materials, Collections, Learning Paths and Workflows are space-limited, whereas Content Providers and Users are shared across all spaces.

Motivation and context

Part of the mTeSS-X OSCARS project.

Screenshots

image

image

Checklist

  • I have read and followed the CONTRIBUTING guide.
  • I confirm that I have the authority necessary to make this contribution on behalf of its copyright owner and agree
    to license it to the TeSS codebase under the
    BSD license.

```
Could not log "request.rsolr" event. NameError: uninitialized constant Sunspot::Rails::LogSubscriber::BOLD
```
Allow icons to be re-coloured per-theme. Allow theme to be selected on space
existing_items_method: :administrators %>

<div class="form-group">
<%= f.submit ( f.object.new_record? ? "Register" : "Update") + " space", class: 'btn btn-primary' %>
Copy link
Contributor

Choose a reason for hiding this comment

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

This should go through i18n

Copy link
Contributor

@cwant cwant left a comment

Choose a reason for hiding this comment

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

Just a brief scan (view mostly) to locate spots where I saw some strings that don't go through i18n.

Create space
<% end %>
<% end %>
<%= info_button("What are spaces?", hide_text: true) do %>
Copy link
Contributor

Choose a reason for hiding this comment

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

This (and following line) should go through i18n

<div class="text-right my-3">
<% if policy(:space).create? %>
<%= link_to new_space_path, class: 'btn btn-primary' do %>
Create space
Copy link
Contributor

Choose a reason for hiding this comment

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

This should go through i18n

@@ -0,0 +1,3 @@
<%= info_box('What are space?') do %>
Copy link
Contributor

Choose a reason for hiding this comment

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

i18n

@@ -0,0 +1,12 @@
<%- model_class = Space -%>
<div class="page-header">
<%= page_title(t('.title', default: [:'helpers.titles.new', 'New %{model}'], model: model_class.model_name.human.titleize.downcase)) %>
Copy link
Contributor

Choose a reason for hiding this comment

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

i18n

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