Skip to content

Conversation

@LeviCameron1
Copy link
Collaborator

Rationale

Layout Editor for the cage UI project

Related Pull Requests

Changes

  1. Created a new module CageUI for the project
  2. Created layout editor page for room creation
  3. Created permissions for the layout editor
  4. Created tables required for layout editor

… rack

This also fixed the bug with some mods not getting updated across certain scenarios
@LeviCameron1 LeviCameron1 requested a review from dnicolalde May 20, 2025 15:24
Copy link
Collaborator

@dnicolalde dnicolalde left a comment

Choose a reason for hiding this comment

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

lets check the version number, we are probably going to deploy it in 24.11, your .sql would have to change name cage-24.000-24.011 to match the current version of LabKey that we are running?

Copy link
Collaborator

Choose a reason for hiding this comment

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

copyright should be for WNPRC

Copy link
Collaborator

Choose a reason for hiding this comment

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

Add copyright to WNPRC

@@ -0,0 +1,25 @@
<%
/*
* Copyright (c) 2025 Board of Regents of the University of Wisconsin System
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should be the copyright text.

@dnicolalde
Copy link
Collaborator

Let's wait until tomorrow to merge this changes. I want to deploy the changes to necropsy before we merge this branch.

@dnicolalde dnicolalde merged commit 007e6f9 into release24.11-SNAPSHOT Jun 5, 2025
8 of 11 checks passed
@dnicolalde dnicolalde deleted the 24.11_fb_cageui_layout branch June 5, 2025 15:01
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.

3 participants