Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: Disallow user to check NCA / Outputs tabs without mapping data #143

Open
Gero1999 opened this issue Dec 16, 2024 · 0 comments · May be fixed by #152
Open

Bug: Disallow user to check NCA / Outputs tabs without mapping data #143

Gero1999 opened this issue Dec 16, 2024 · 0 comments · May be fixed by #152
Assignees
Labels
bug Something isn't working

Comments

@Gero1999
Copy link
Collaborator

Description

When user has not mapped the data (define column names and accepted) it should not be able to go through other tabs (NCA or Outputs) as it can be confusing for them to see errror messages afterwards. Instead, those tabs should remain disable and perhpaps even show a notification

To reproduce

Intiiate the app and click NCA or Outputs tab. In the second one you won't find results but instead just error messages.

Expected behaviour

Disability to go to Ouptuts or NCA without mapping.
Notification with instructions on what to do (mapping the data)

@Gero1999 Gero1999 added the bug Something isn't working label Dec 16, 2024
@js3110 js3110 marked this as a duplicate of #142 Dec 16, 2024
@js3110 js3110 moved this to Backlog in aNCA Dec 16, 2024
@js3110 js3110 self-assigned this Jan 3, 2025
@js3110 js3110 linked a pull request Jan 3, 2025 that will close this issue
2 tasks
@js3110 js3110 linked a pull request Jan 3, 2025 that will close this issue
2 tasks
@js3110 js3110 moved this from Backlog to Needs review in aNCA Jan 3, 2025
@js3110 js3110 moved this from Needs review to In review in aNCA Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: In review
Development

Successfully merging a pull request may close this issue.

2 participants