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

Update single-scripture-rcl version for #546 #558

Merged
merged 1 commit into from
Oct 13, 2023

Conversation

kintsoogi
Copy link
Contributor

Describe what your pull request addresses

  • Fixes scripture not reloading on update from master. Issue 546

Test Instructions

  • Make edits to a Scripture Card
  • Make to the same resource on another account
  • Update the scripture card changes, verifying that content reloads

@codecov
Copy link

codecov bot commented Oct 11, 2023

Codecov Report

Merging #558 (72aec0d) into develop (855e119) will decrease coverage by 0.27%.
Report is 68 commits behind head on develop.
The diff coverage is 64.20%.

@@             Coverage Diff             @@
##           develop     #558      +/-   ##
===========================================
- Coverage    51.72%   51.46%   -0.27%     
===========================================
  Files           43       43              
  Lines         1301     1296       -5     
===========================================
- Hits           673      667       -6     
- Misses         628      629       +1     
Files Coverage Δ
pages/_app.js 100.00% <ø> (ø)
src/common/constants.js 100.00% <100.00%> (ø)
src/components/Drawer.js 33.33% <100.00%> (ø)
src/components/Header.js 68.18% <100.00%> (ø)
src/context/AuthContext.js 45.71% <100.00%> (ø)
src/context/StoreContext.js 85.71% <100.00%> (+0.86%) ⬆️
src/hooks/useWindowEvent.js 83.33% <83.33%> (ø)
src/utils/lexiconHelpers.js 23.72% <0.00%> (ø)
src/components/Layout.js 73.33% <60.00%> (ø)
src/components/PopoverComponent.js 77.77% <77.77%> (ø)
... and 6 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@kintsoogi kintsoogi force-pushed the bugfix-kintsoogi-546-scriptureReload branch from 1b6f5fe to 72aec0d Compare October 12, 2023 19:33
@kintsoogi kintsoogi marked this pull request as ready for review October 12, 2023 19:34
@kintsoogi kintsoogi changed the title Temporary beta for deploy preview Update single-scripture-rcl version Oct 12, 2023
@kintsoogi kintsoogi changed the title Update single-scripture-rcl version Update single-scripture-rcl version for #546 Oct 12, 2023
Copy link
Contributor

@PhotoNomad0 PhotoNomad0 left a comment

Choose a reason for hiding this comment

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

LGTM

@PhotoNomad0 PhotoNomad0 merged commit d431e66 into develop Oct 13, 2023
9 of 10 checks passed
@delete-merged-branch delete-merged-branch bot deleted the bugfix-kintsoogi-546-scriptureReload branch October 13, 2023 13:17
PhotoNomad0 added a commit that referenced this pull request Dec 6, 2023
* Replace hard coded card names in merge dialog to resource titles

* Replace BranchMergerContext with state in app context

* Update single-scripture-rcl version for #546 (#558)

* Update issue templates

* Add button to TSV Resource Cards to Add another TSV resource

* Moved add button in TWL to actions column in table

* Feature 559 / support front matter (#561)

* added front to demo

* update dependencies

* update dependencies

* update dependencies

* update dependencies

* add error message handling

* update dependencies

* update dependencies

* update dependencies

* update dependencies

* update dependencies

* update dependencies

* update dependencies

* update dependencies

* update dependencies

* update dependencies

* update dependencies

* update dependencies

---------

Co-authored-by: PhotoNomad0 <[email protected]>

* Save to DCS on TSV Add & Navigate to new TSV Row (#571)

* Changes to save new TSV row to DCS on add

* Changes to navigate to newly created tsv row

* Only generate Tsv data if resource is tsv

* Change const, clarify ternary, update scrip-tsv version

* feature 524 / added alignment reset and multi-select. (#576)

* added alignment reset and multi-select.

* update dependencies

* update dependencies

---------

Co-authored-by: PhotoNomad0 <[email protected]>

* Update version of scripture-tsv to 0.2.1 (#584)

* Fix for 500 errors (#583)

Updated versions of gitea-react-toolkit and single-scripture-rcl

Co-authored-by: PhotoNomad0 <[email protected]>

* Feature kintsoogi 484 delete tsv (#587)

* Add delete tsv functionality and pure setItemIndex

* Small fix for newQuote

* update dependencies

---------

Co-authored-by: PhotoNomad0 <[email protected]>

* upgrades translation-helps-rcl to 3.5.13

* Display add button in header for TWL when no items (#597)

- Display add button in header for TWL when no items
- Do not display delete button if no items in tsv resource card

* Updated version of scripture-resources-rcl

Fixes problem where empty last tsv item in book throws errors

* fixes build error while building with node >= 18

__This is a temporary fix__

Any machine building with node >= 18 runs into this issue:
unfoldingWord/translation-helps-rcl#121

I've modified the build scripts to use the set-env.sh scripts
to determine what the NODE_OPTIONS should be when building.

* Bump version to 2.2.0

* qaFail 602 / Save button is enabled when edit box is clicked (#608)

update single-scripture-rcl

* Bugfix 570 / Fix highlighting errors with right quote in Greek Word (#604)

fixes for highlighting right quotes in original languages

---------

Co-authored-by: kintsoogi <[email protected]>
Co-authored-by: Kintsoogi <[email protected]>
Co-authored-by: elsylambert <[email protected]>
Co-authored-by: PhotoNomad0 <[email protected]>
Co-authored-by: Noah Harvey <[email protected]>
Co-authored-by: Noah Harvey <[email protected]>
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