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

feat: similar authors #567

Merged
merged 74 commits into from
Jun 22, 2023
Merged

feat: similar authors #567

merged 74 commits into from
Jun 22, 2023

Conversation

djdembeck
Copy link
Collaborator

@djdembeck djdembeck commented Feb 27, 2023

  • Update tests and test data
  • Add cases with and without similar on html.
  • Make sure author has name

/sims doesn't seem to work anymore. Book similarity might need to be scraped, which isn't optimal.

@djdembeck djdembeck changed the title feat: similar feat: similar authors Jun 21, 2023
@github-actions
Copy link

github-actions bot commented Jun 21, 2023

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
98.63% (-0.08% 🔻)
1007/1021
🟢 Branches
96.93% (-0.71% 🔻)
253/261
🟢 Functions
98.97% (+0.03% 🔼)
192/194
🟢 Lines
98.83% (-0.09% 🔻)
928/939
Show files with reduced coverage 🔻
St.
File Statements Branches Functions Lines
🟢
... / shared.ts
97.62% (+0.06% 🔼)
94.74% (-5.26% 🔻)
100%
97.56% (+0.06% 🔼)
🟢
... / ScrapeHelper.ts
94.64% (-0.91% 🔻)
60% (-6.67% 🔻)
100%
94.55% (-1.01% 🔻)

Test suite run success

283 tests passing in 25 suites.

Report generated by 🧪jest coverage report action from d18f7ab

@laxamentumtech laxamentumtech deleted a comment from github-actions bot Jun 22, 2023
@djdembeck djdembeck marked this pull request as ready for review June 22, 2023 20:01
@djdembeck djdembeck added the enhancement New feature or request label Jun 22, 2023
@djdembeck djdembeck merged commit 87d6114 into develop Jun 22, 2023
6 checks passed
@djdembeck djdembeck mentioned this pull request Jun 22, 2023
djdembeck added a commit that referenced this pull request Jun 22, 2023
* feat: similar authors (#567)

* refactor(helper): ♻️ allow re-using asin10 regex by removing anchors

* build: ➕ add `redoc-cli` as dev dep for generating html docs

* docs: 📝 update api spec and html

adds example responses; remove unused fields that aren't currently returned for author; fix rating showing number and  releaseDate using date instead of date-time
@djdembeck djdembeck deleted the feat-similar branch June 22, 2023 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant