DMC/scipro
7
0
Fork 0
Code Issues 33 Pull Requests 4 Actions Projects 1 Wiki Activity

Fix localizer warning on the finishing up tab for each author #101

Open
ansv7779 wants to merge 1 commits from reflection_not_submitted into develop
pull from: reflection_not_submitted
merge into: DMC:develop
Conversation 1 Commits 1 Files Changed 1 +7 -1
ansv7779 commented 2025-02-05 13:32:02 +01:00
Owner
Copy Link

Fixes #48

The getReflectionText method was calling getString when no reflection has been submitted. This is not a relevant case for the editing form since it can not be accessed when there is no reflection. Inlined the method call and removed the non-submitted case, the default will be an empty string.

How to test

  1. Log in as a supervisor
  2. Open a project that has a Daisy connection (identifieris non-null on the Project)
  3. Go to the "Finishing up" tab
  4. Go to the tab for the author with no reflection submitted
  5. See that no warning is logged
Fixes #48 The `getReflectionText` method was calling `getString` when no reflection has been submitted. This is not a relevant case for the editing form since it can not be accessed when there is no reflection. Inlined the method call and removed the non-submitted case, the default will be an empty string. ## How to test 1. Log in as a supervisor 2. Open a project that has a Daisy connection (`identifier`is non-null on the `Project`) 3. Go to the "Finishing up" tab 4. Go to the tab for the author with no reflection submitted 5. See that no warning is logged
ansv7779 added 1 commit 2025-02-05 13:32:02 +01:00
Fix localizer warning on the finishing up tab for each author
All checks were successful
Build and test / build-and-test (push) Successful in 15m47s
Details
Deploy to branch.dsv.su.se / deploy (pull_request) Successful in 2m13s
Details
8d6faa8ee7
gitea-actions commented 2025-02-05 13:34:16 +01:00
First-time contributor
Copy Link

Deployed to https://scipro-reflectionnotsubmitted.branch.dsv.su.se

Deployed to https://scipro-reflectionnotsubmitted.branch.dsv.su.se
All checks were successful
Build and test / build-and-test (push) Successful in 15m47s
Details
Deploy to branch.dsv.su.se / deploy (pull_request) Successful in 2m13s
Details
This pull request doesn't have enough required approvals yet. 0 of 1 official approvals granted.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin reflection_not_submitted:reflection_not_submitted
git checkout reflection_not_submitted
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
discussion
duplicate
This issue or pull request already exists
enhancement
New feature
frozen
The issue is not solved, but it will be investigated in the future it the issue arises again
help wanted
Need some help
invalid
Something is wrong
new
To be discussed at the next meeting
po
approved
po
needed
question
More information is needed
size
1
size
2
size
3
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DMC/scipro#101
No description provided.
Delete Branch "reflection_not_submitted"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?