Conversation
Pull from imageomics catalog [PR 92](Imageomics/catalog#92) * extract normalizeTag into src/normalizeTag.js * add tests for normalizeTag * extract filter and sort logic into src/filterAndSort.js * add tests for filterItems and sortItems * extract HF additional repo dedup into src/filterNewAdditionalEntries.js * add tests for filterNewAdditionalEntries --------- Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com> (cherry picked from commit 4f1e2eb780a1ca181d85f43ab6734e1ba45fe8be)
Pull from imageomics catalog [PR 94](Imageomics/catalog#94) * Use Catalog App for tag-scan PR * Add documentation for creating a GitHub App for authentication (cherry picked from commit cffb80cbeed5887d1dd910815484cacf38b42ce3)
Pull from imageomics catalog [PR 97](Imageomics/catalog#97) * Move code platform definition to config Add functions to define API URLs and ribbon values based on config-defined platform * Add check for valid platform, throw informative error (config-based) (cherry picked from commit 9f0afdda5f8c07cb568bfe55c211b89672a41ecb)
Pull from imageomics catalog [PR 101](Imageomics/catalog#101) (cherry picked from commit 47aeae0ccab53e367bf582a19da8ff4e3f552727)
Pull from imageomics catalog [PR 102](Imageomics/catalog#102) This was a suggestion from Copilot to improve log messaging; displayName is set from 'defineRibbonVals.js', but the import was missed
Pull from imageomics catalog [PR 104](Imageomics/catalog#104) * Simplify intro, move detailed personalization and generation documentation to docs directory * Replace PAT description with App-based authentication and automation pointer * Revise intro paragraph for increased generality, emphasize template aspect earlier to encourage reuse (cherry picked from commit 3e0757d4d73f2daa0ac01ac6a959cd43fe7956aa)
Pull from imageomics catalog [PR 103](Imageomics/catalog#103) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> (cherry picked from commit a1963abcaedc440c683d7bc5e7b25dac7eb7fba8)
egrace479
reviewed
May 22, 2026
Member
egrace479
left a comment
There was a problem hiding this comment.
A couple suggested re-writes to make it ABC-specific and include the newer HF manual addition functionality.
Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>
egrace479
approved these changes
May 22, 2026
Member
egrace479
left a comment
There was a problem hiding this comment.
Tested that it ran locally too.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
feat: decouple core features from DOM for testing
Use Catalog App for tag-scan PR
Move code platform definition to config
Fix module path
Add import for getPlatformDisplay function
Reorganize catalog documentation
[Tag Scan] New tags detected — review tag-groups.js
README HOW TO USE TEMPLATE & Intro section changes are not applied to the ABC catalog README.