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

PDF Generator update #361

Merged
merged 159 commits into from
Sep 12, 2024
Merged

Conversation

DanielC-N
Copy link
Contributor

added a button on the top right corner to generate a PDF from one or more project with different downloaded resources.

danielc-n and others added 30 commits January 16, 2024 15:20
- added a 'factoryCreateContent.js'  file to help factorize some code for 'createObsContent', 'createVersificationUSFM' and 'createAudioVersification' in 'saveProjetcsMeta.js'

- fixing some typos, fixing lint
… to search and download a SB as a resource in a module
- auto load a book on the opening of a project
- auto save after a modification
- fixed a bit of typos and code
…lso edited the icon for Juxta mode (it's visible now) you can create a new juxta project
…, 'double click' to modify a chunck, 'enter' to end the modification of a chunck. And added seemless navigation between Juxtalinear editor and the BibleEditorX. Navigating through verses and chapters with the menu bar will load the corresponding sentence and vice versa.
… the user : to know if the help pop has already been opened once
@DanielC-N
Copy link
Contributor Author

actions/upload-artifact@v1 is now deprecated and unusable.
Today I received an error from github actions : "Error: This request has been automatically failed because it uses a deprecated version of **actions/upload-artifact: v1**. Learn more: https://github.blog/changelog/2024-02-13-deprecation-notice-v1-and-v2-of-the-artifact-actions/"

danielc-n added 3 commits September 10, 2024 14:20
[1/4] Resolving packages...
success Already up-to-date.
Done in 0.65s. so I'm trying to make it verbose to detect the problem. Also, I authorized the overwrite of the artifacts
@selvabenz
Copy link

selvabenz commented Sep 11, 2024

@DanielC-N, the PR looks good. there are some suggestions I have.

  1. If you could keep the Scribe UI standard, the whole UI would look consistent. @arpitjacob can help you with the UI standards
  2. Reset parameters is inactive when I want to reset
  3. Start page side, user may understand Right and Left easily than Recto and Verso. (just a suggestion)
  4. In "Your file name here", unable to give any space and special character. if it is kept intentionally, ignore this
  5. Add validation message for the "Choose export Folder" and "Your file name here" if "Choose export Folder" not chosen or "Your file name here" name typed
  6. Add validation message for OBS start and end
  7. If "Your file name here" is empty it is taking random alphabet and generates the PDF
  8. On one instance I did not select "show page number but in the message area it shows "Generate Page Numbers"
  9. Add complete or failed message when the printing is done or failed
  10. Sample PDF of the Juxta, OBS and Text translation
    Test.pdf
    b.pdf
    t.pdf

All the other functions are looking good.

@DanielC-N
Copy link
Contributor Author

DanielC-N commented Sep 11, 2024

  1. If you could keep the Scribe UI standard, the whole UI would look consistent. @arpitjacob can help you with the UI standards ❌
  2. Reset parameters is inactive when I want to reset ✅
  3. Start page side, user may understand Right and Left easily than Recto and Verso. (just a suggestion) ❌
  4. In "Your file name here", unable to give any space and special character. if it is kept intentionally, ignore this => This is intentional but I added a notice ✅
  5. Add validation message for the "Choose export Folder" and "Your file name here" if "Choose export Folder" not chosen or "Your file name here" name typed ✅
  6. Add validation message for OBS start and end ❓
  7. If "Your file name here" is empty it is taking random alphabet and generates the PDF => I put a string of random words ✅
  8. On one instance I did not select "show page number but in the message area it shows "Generate Page Numbers" ✅
  9. Add complete or failed message when the printing is done or failed ✅

@selvabenz
Copy link

Thanks, @DanielC-N for updating the PR.

I have made a video to explain point no 6 Add validation message for OBS start and end ❓
https://www.awesomescreenshot.com/video/31440031?key=012d2af149814967156640a5b86a9140

@DanielC-N
Copy link
Contributor Author

@selvabenz I've hidden the print button on audio projects.
After the discussion for the OBS. Nothing was changed.

@selvabenz
Copy link

@vipinpaul, all the necessary corrections are made by @DanielC-N, we can go for a merge.

@vipinpaul vipinpaul merged commit 7745fd3 into bible-technology:development Sep 12, 2024
2 checks passed
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.

4 participants