Leftmost: As pointed out this is common issue with identifier editor and regex detection in general, i believe rather than hacking around the regex detection, we should have simple "turn off" checkbox to disable this behavior.
Leftmost
Shubh, there are some issues with that, yeah, but I think they're exacerbated by taking automatic action when we detect an ISBN and we should probably be more liberal in deciding when two ISBNs are equal. (If we have 978-0-15-603521-7, we shouldn't add 97806035217.) And I think rather than putting the responsibility on users to disable the behavior, we should probably be checking if something is a valid ISBN rather than
relying on regex when ISBNs aren't a regular language.
cloverfield joined the channel
Shubh
I think we can verify the last digit of ISBN using checksum algorithm when detecting for identifier type
but that doesn't solve the issue for other types we might have in future...
asymmentric joined the channel
asymmentric has quit
asymmentric joined the channel
CatQuest joined the channel
CatQuest has quit
CatQuest joined the channel
asymmentric has quit
asymmentric joined the channel
kellnerd joined the channel
asymmentric has quit
monkey: i'm facing some issues with adding bookbrainz orm from github branch, any idea on how prabal did it?
But generally, if you have both repos locally, you can run `npm link` in the ORM and then `npm link bookbrainz-data` in bookbrainz-site to use your local version
(note that if you install nay package after that you'll need to re-link)
asymmentric joined the channel
CandyBrain joined the channel
asymmentric11 joined the channel
asymmentric11 has quit
asymmentric has quit
CandyBrain has left the channel
agatzk has quit
agatzk joined the channel
kellnerd has quit
agatzk has quit
agatzk joined the channel
ShivamAwasthi joined the channel
ShivamAwasthi
Shubh are you still facing the issue? I always forget to install the dependencies before linking. 👀