Skip to content

feature/KBDEV-1502 cosmic loader update#195

Open
sshugsc wants to merge 7 commits into
developfrom
feature/KBDEV-1502-cosmic-loader-update
Open

feature/KBDEV-1502 cosmic loader update#195
sshugsc wants to merge 7 commits into
developfrom
feature/KBDEV-1502-cosmic-loader-update

Conversation

@sshugsc

@sshugsc sshugsc commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

KBDEV-1502

  • update cosmic loader

@sshugsc sshugsc self-assigned this Apr 15, 2026
@mathieulemieux

Copy link
Copy Markdown
Member

@sshugsc, do you have the upload logs from you tests?

@sshugsc

sshugsc commented Apr 16, 2026

Copy link
Copy Markdown
Contributor Author

@mathieulemieux logs could be found at jira ticket KBDEV-1502

@mathieulemieux

Copy link
Copy Markdown
Member

@sshugsc, I'll need either the cosmic files, or the cosmic email/password.

@sshugsc

sshugsc commented Apr 21, 2026

Copy link
Copy Markdown
Contributor Author

@mathieulemieux sent by rocket chat

Comment thread src/hgnc/index.js Outdated
Comment thread src/entrez/util.js
Comment thread src/cosmic/resistance.js Outdated
Comment thread src/cosmic/resistance.js
}
} else {
logger.info('Cannot delete previously existing statements when errors were encoutered');
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Errors happened when testing, indeed. Seems bad to not beeing able to delete any statements because of a single error. I'm wondering if we can think of something better.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yea, that would be ideal but we don't have a way to match the new created statements with the original statements. It would be difficult to decide which statement should not be deleted. @elewis2 any thoughts?

Comment thread src/cosmic/resistance.js
@mathieulemieux

Copy link
Copy Markdown
Member

@elewis2 , should we implement tests here or it's out of scope for this ticket/loader?

@sshugsc sshugsc requested a review from mathieulemieux April 22, 2026 19:23
Comment thread src/cosmic/resistance.js
@elewis2

elewis2 commented May 6, 2026

Copy link
Copy Markdown
Collaborator

@elewis2 , should we implement tests here or it's out of scope for this ticket/loader?

Yes, tests should be included in this work.

@sshugsc sshugsc requested a review from mathieulemieux May 13, 2026 20:54
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.

3 participants