Skip to content

SIunit frontend for backend 217#136

Draft
herrdivad wants to merge 32 commits into
masterfrom
SIunit-frontend-for-backend-217
Draft

SIunit frontend for backend 217#136
herrdivad wants to merge 32 commits into
masterfrom
SIunit-frontend-for-backend-217

Conversation

@herrdivad

Copy link
Copy Markdown
Collaborator

No description provided.

@herrdivad herrdivad requested a review from StarmanMartin March 27, 2026 09:36
@herrdivad

herrdivad commented Mar 27, 2026

Copy link
Copy Markdown
Collaborator Author

Frontend:

  • -- Auto description that it was a conversion from SI units packages
  • -- Adding multiple assignments for one and the same unit to multiple columns or tables
  • -- Modal for assigning x|y units directly in the data tables tab

Backend:

  • -- Standard units to extractor?
  • -- Check how units are saved and what happens if one file for the same reader and profile uses "m" and the other "cm"

@herrdivad

Copy link
Copy Markdown
Collaborator Author

@herrdivad herrdivad linked an issue Apr 8, 2026 that may be closed by this pull request
const hasHiddenAssignments = displayState === "hidden";
const variant = isOpen
? "outline-info"
: (hasHiddenAssignments ? "warning" : "outline-success");

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Try changing to outline-warning.

</Table>
)}
</Card.Body>
<Modal

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

refactor getCurrentEditorAssignment & customConversionEditor checks.
E.g. const editorAssignment = getCurrentEditorAssignment(); outside of return and
{editorAssignment && ()}
getCurrentEditorAssignment already checks customConversionEditor.

herrdivad added 4 commits May 28, 2026 09:04
# Conflicts:
#	build/admin/index.js
#	src/js/components/admin/form/TableForm.js
#	src/js/components/admin/form/controllComponents/DataTables.js
#	src/js/components/admin/form/controllComponents/FormNavigator.js
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.

dynamic and flexible unit list and conversions

3 participants