bible-technology / scripture-burrito

Scripture Burrito Schema & Docs 🌯
http://docs.burrito.bible/
MIT License
21 stars 13 forks source link

bug fixes for schema/scripture_flavor_type.schema.json #311

Closed theNerd247 closed 2 months ago

theNerd247 commented 8 months ago

Some of the files refered to were removed or renamed. These changes were not reflected in the consuming file (modified in this commit) and caused consuming tools to break. I assume these changes were originally intended to be reflected in this schema. The relevant details from the git log of which files were changed are noted below:

Further, I've kept the key "bookScope" even though the filename is "scope.schema.json". I assume the file names used are semantic, however I did not want to break backwards compatibility. I am willing to make the change to update the schema from "bookScope" to "scope" if the semantics should be reflected between the schema definition and the names of the files used.

I've also removed the extraneous comma as noted in issue #310

Relevant Commit Logs:

commit 69e2f8a1d3ee29c149865baee991b48776154f73 Author: Mark Howe mvahowe@gmail.com Date: Thu Apr 2 17:26:29 2020 +0200 D schema/scripture/braille_scripture_publication.schema.json R097 schema/scripture/print_publication.schema.json schema/scripture/typeset_scripture.schema.json

commit 847f0eb2f0c8191165e38280d3c52558c3446e23 Author: Mark Howe mvahowe@gmail.com Date: Wed Feb 26 15:17:59 2020 +0000 R100 schema/book_scope.schema.json schema/scope.schema.json

theNerd247 commented 8 months ago

I’m not sure. The git log shows that the brail_scripture_publication file was deleted and the one you referenced was added - but the file contents had a significant change. I’m not knowledgeable enough of the system to make this decision. Maybe Mark Howe knows?

On Nov 14, 2023, at 5:42 PM, Richard Mahn @.***> wrote:

@richmahn commented on this pull request.

In schema/scripture_flavor_type.schema.json https://github.com/bible-technology/scripture-burrito/pull/311#discussion_r1393400261:

@@ -22,10 +22,7 @@ "$ref": "scripture/audio_translation.schema.json" }, {

  • "$ref": "scripture/print_publication.schema.json"
  • },
  • {
  • "$ref": "scripture/braille_scripture_publication.schema.json"
  • "$ref": "scripture/typeset_scripture.schema.json" Shouldn't this be
                        "$ref": "scripture/typeset_scripture.schema.json"
                    },
                    {
                        "$ref": "scripture/embossed_braille_scripture.schema.json"

— Reply to this email directly, view it on GitHub https://github.com/bible-technology/scripture-burrito/pull/311#pullrequestreview-1730949621, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAG6ND5NKUFPXELRPXMSSW3YEPXXBAVCNFSM6AAAAAA7GTB7PKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOMZQHE2DSNRSGE. You are receiving this because you authored the thread.

jtauber commented 5 months ago

I do not believe this file is being used. No tests were failing because of the extra comma.