Skip to content

PoC new inventory table - #7422

Open
aignatov-bio wants to merge 2 commits into
scinote-eln:features/inventory-table-revampfrom
aignatov-bio:ai-sci-10537-poc-new-inventory-table
Open

PoC new inventory table#7422
aignatov-bio wants to merge 2 commits into
scinote-eln:features/inventory-table-revampfrom
aignatov-bio:ai-sci-10537-poc-new-inventory-table

Conversation

@aignatov-bio

Copy link
Copy Markdown
Contributor

What was done

PoC new inventory table

});

FULL_VIEW_MODAL.on('hidden.bs.modal', function() {
return

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing semicolon semi

repositoryTable.attr('data-name-column-id', $(this).data('name-column-id'));
repositoryTable.attr('data-stock-management', $(this).data('data-stock-management'));
repositoryContainer.html(repositoryTable);
//repositoryContainer.html(repositoryTable);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Expected space or tab after '//' in comment spaced-comment

);
updateFullViewRowsCount(tableContainer.attr('data-assigned-items-count'));
window.mountRepositoryTable();
return

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing semicolon semi

function renderSimpleTable(tableContainer) {
console.log(tableContainer.attr('data-element'))
window.mountRepositoryTable(tableContainer.attr('data-element'));
return

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing semicolon semi

}

function renderSimpleTable(tableContainer) {
console.log(tableContainer.attr('data-element'))

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing semicolon semi
Unexpected console statement no-console


render rows_view
repository_rows = Lists::RepositoryRowsService.new(@repository, params, current_user, @my_module).call
render json: repository_rows, each_serializer: Lists::RepositoryRowSerializer, user: current_user, my_module: @my_module,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Layout/LineLength: Line is too long. [125/120]

user: current_user,
column: stock_cell.repository_column,
repository: repository,
options: {reminders_enabled: reminders_enabled}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[Correctable] Layout/SpaceInsideHashLiteralBraces: Space inside { missing.
[Correctable] Layout/SpaceInsideHashLiteralBraces: Space inside } missing.

if has_stock_management
stock_cell = object.repository_cells.find { |cell| cell.value_type == 'RepositoryStockValue' }
stock_column = repository.repository_columns.find_by(data_type: 'RepositoryStockValue')
col_key = 'col_' + stock_column.id.to_s

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[Correctable] Style/StringConcatenation: Prefer string interpolation to string concatenation.

@@ -0,0 +1,109 @@
# frozen_string_literal: true

class Lists::RepositoryRowSerializer < ActiveModel::Serializer

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[Correctable] Style/ClassAndModuleChildren: Use nested module/class definitions instead of compact style.

# .group('repository_sort_cells.repository_row_id')
# else
# cells
# .select("repository_sort_cells.repository_row_id, #{sorting_data_type::SORTABLE_COLUMN_NAME} AS value")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Layout/LineLength: Line is too long. [125/120]

@scinote-eln scinote-eln deleted a comment from hound Bot Jan 5, 2026
@scinote-eln scinote-eln deleted a comment from hound Bot Jan 7, 2026
@scinote-eln scinote-eln deleted a comment from hound Bot Jan 7, 2026
@scinote-eln scinote-eln deleted a comment from hound Bot Jan 7, 2026
@scinote-eln scinote-eln deleted a comment from hound Bot Jan 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant