Skip to content

Conversation

@nitrosx
Copy link
Member

@nitrosx nitrosx commented Aug 20, 2025

Description

This PR introduces all the changes in the frontend config that are needed for the PRs listed in the next section.

Motivation

FE PRs (1975)[https://github.com/SciCatProject/frontend/pull/1975] and (1976)[https://github.com/SciCatProject/frontend/pull/1976] requires some changes to the FE configuration provided by the BE.

Changes:

  • config/frontend.config.json

Tests included

  • Included for each change/fix?
  • Passing?

Documentation

  • swagger documentation updated (required for API changes)
  • official documentation updated

official documentation info

Non yet

@Junjiequan Junjiequan force-pushed the new_frontend_config branch from 5e7abbf to b3bc4de Compare August 27, 2025 13:31
Copy link
Member

@minottic minottic left a comment

Choose a reason for hiding this comment

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

it might be a bit hard to modify later as the order field is differnet from the list order. Would it help to try and keep the order consitent (i.e. moving block with order: 1 on top etc)

@nitrosx
Copy link
Member Author

nitrosx commented Nov 27, 2025

@minottic I agree with you that the configuration should be in the order in which the elements are shown, although I favor an explicit property that define the order.

ordered dataset columns
@nitrosx nitrosx requested a review from a team as a code owner November 27, 2025 13:22
@nitrosx
Copy link
Member Author

nitrosx commented Nov 27, 2025

@minottic I changed the list of columns for the datasets list to be in order.

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.

4 participants