Skip to content

Commit 58a9190

Browse files
authored
fix(Dr. Rai Reports): Tie Sidebar to selected period (#5683)
**Story card:** [sc-16421](https://app.shortcut.com/simpledotorg/story/16421/fixes-to-dr-rai-mvp) ## Because With #5675, tying the sidebar to the current period no longe rmakes sense. ## This addresses Ensuring the sidebar works with the selected period instead of the current period ## Test instructions create a future plan
1 parent 0634f35 commit 58a9190

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

app/components/dashboard/dr_rai_report.html.erb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@
6969
<% end %>
7070
</div>
7171
<% unless is_lite_version? %>
72-
<div class="sidepanel bs-canvas bs-canvas-right position-fixed h-100" tabindex="-1" id="dr-rai--sidebar" data-period="<%= current_period.value %>" data-region="<%= region.slug %>">
72+
<div class="sidepanel bs-canvas bs-canvas-right position-fixed h-100" tabindex="-1" id="dr-rai--sidebar" data-period="<%= selected_period.value %>" data-region="<%= region.slug %>">
7373
<div class="header">
7474
<p>Create action</p>
7575
<button class="bs-canvas-close float-left close" aria-label="Close">

0 commit comments

Comments
 (0)