verdnatura/salix
verdnatura
/
salix
3
5
Fork 2
Code Issues Pull Requests 70 Releases Wiki Activity

feat: refs #8657 add application status report and related templates #3490

Open
alexm wants to merge 8 commits from 8657-app-status-pdf into dev
pull from: 8657-app-status-pdf
merge into: verdnatura:dev
Conversation 1 Commits 8 Files Changed 4 +34 -1
alexm commented 2025-02-24 11:26:54 +00:00
Member
Copy Link
No description provided.
alexm added 1 commit 2025-02-24 11:26:55 +00:00
gitea/salix/pipeline/pr-master There was a failure building this commit Details
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
75b3be8b79 feat: refs #8657 add application status report and related templates
alexm commented 2025-02-24 11:27:04 +00:00
Author
Member
Copy Link

image

![image](/attachments/2bd657e5-2643-47db-af11-b65afe06e4d3)
image.png
295 KiB
alexm reviewed 2025-02-24 11:27:27 +00:00
loopback/common/methods/application/status.js
@ -14,6 +16,7 @@ module.exports = Self => {
Self.status = async() => {
await Self.rawSql(`SELECT TRUE`);
await new Report('application-status').toPdfStream();
alexm commented 2025-02-24 11:27:27 +00:00
Author
Member
Copy Link

Sense el toPdfStream no dona un 500

Sense el toPdfStream no dona un 500
alexm changed target branch from master to dev 2025-02-24 11:35:33 +00:00
alexm added 1 commit 2025-02-24 11:38:38 +00:00
gitea/salix/pipeline/pr-dev Something is wrong with the build of this commit Details
9fad296757 Merge branch 'dev' into 8657-app-status-pdf
alexm added 2 commits 2025-02-24 12:07:04 +00:00
alexm added 1 commit 2025-02-24 13:28:38 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
acf0f82d85 fix: refs #8657 test await printer server
alexm added 1 commit 2025-02-24 13:43:58 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
3ed1d2e163 Merge branch 'dev' into 8657-app-status-pdf
alexm added 1 commit 2025-02-25 06:15:59 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
f2421c806a Merge branch 'dev' into 8657-app-status-pdf
alexm added 1 commit 2025-02-26 06:51:55 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
b660c82ffa Merge branch 'dev' into 8657-app-status-pdf
alexm requested review from juan 2025-02-26 06:51:58 +00:00
juan requested changes 2025-02-27 08:15:31 +00:00
juan left a comment
Owner
Copy Link

Intentar solucionar el problema de los test sin setInterval()

Intentar solucionar el problema de los test sin `setInterval()`
back/tests-helper.js
@ -36,0 +37,4 @@
const MAX_ATTEMPTS = 10;
let attempts = 0;
return new Promise((resolve, reject) => {
const intervalo = setInterval(async() => {
juan commented 2025-02-27 08:14:49 +00:00
Owner
Copy Link

Las variables deben estar en ingles

Las variables deben estar en ingles
Some checks failed
gitea/salix/pipeline/pr-dev There was a failure building this commit
Required
Details
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
Reviewers
No reviewers
juan
Labels
Clear labels   
CR / Tests passed

Unit tests and e2e passed

  
e2e

Implement e2e

  
Missing tests

Missing tests

  
Needs refactor

   
Simple

Simple check / No CR needed

No Label
CR / Tests passed
e2e
Missing tests
Needs refactor
Simple
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix#3490
No description provided.
Delete Branch "8657-app-status-pdf"

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?