Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: Reference Parsing #391

Merged
merged 6 commits into from
Nov 13, 2023

Conversation

jakedcolbert
Copy link
Contributor

@jakedcolbert jakedcolbert commented Aug 3, 2023

Implements #388

  • Develop utilities for parsing in-text references and generating
    respective HTML content.
  • Develop tests to evaluate effectiveness of developed functions
  • Develop footnote content display component with stack functionality.

@jakedcolbert jakedcolbert changed the title Created scripture-reference.ts file Feature: Reference Parsing #388 Aug 12, 2023
@jakedcolbert jakedcolbert force-pushed the feature/footnote-ref/388 branch 5 times, most recently from 9c73ac4 to b06f276 Compare August 28, 2023 15:28
@jakedcolbert
Copy link
Contributor Author

Ready for review @chrisvire

@jakedcolbert jakedcolbert changed the title Feature: Reference Parsing #388 Feature: Reference Parsing Aug 31, 2023
@chrisvire
Copy link
Member

@jakedcolbert I am going through PRs and noticed this one. Do you believe this is ready?

@jakedcolbert
Copy link
Contributor Author

jakedcolbert commented Oct 25, 2023

@chrisvire Yes, it meets the requirements of #388. It does not handle verse list separators or books with single chapter formats like 'Jude 6'

jakedcolbert added 5 commits November 6, 2023 11:00
* Develop utilities for parsing in text references and generating
  respective HTML content.
* Develop tests to evaluate effectiveneess of developed functions
* Develop footnote content display component with stack functionality.
- Fix expected phrases that would have failed when encountering non
  standard separators.
@chrisvire
Copy link
Member

@jakedcolbert I am confused on what I should be looking for? Is there specific Langauage/Book/Chapter/Verse where I can see this in action?

Copy link
Member

@chrisvire chrisvire left a comment

Choose a reason for hiding this comment

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

👍

@chrisvire chrisvire merged commit 3b3a517 into sillsdev:main Nov 13, 2023
1 check passed
@jakedcolbert jakedcolbert deleted the feature/footnote-ref/388 branch April 18, 2024 17:09
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.

2 participants