CenterForDigitalHumanities / CenterForDigitalHumanities/TPEN-services

adjust property sorting

Open
#490 0 comments 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
JavaScript
Stars
2
Forks
0
PR merge metrics
No merged PRs in 30d

Description

> The synchronous logic to "predict" the RERUM page ID duplicates the logic from `Page.#setRerumId`. This creates maintenance burden - if the ID generation logic changes in the Page class, it must be updated here as well. Consider refactoring to call a shared function or exposing a method on the Page class to handle this logic consistently.

_Originally posted by @Copilot in [#431](https://github.com/CenterForDigitalHumanities/TPEN-services/pull/431/changes#r2608032621)_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.