-
Notifications
You must be signed in to change notification settings - Fork 26
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
MASS SET WORKS’ RECORDING DATES broken with the new relationship editor #807
Comments
I made an attempt: 24cf11e But it does not work:
|
It seems that I have to convert this feature to old school userscript where have to simulate user interactions: click pencil, focus inputs, type in inputs, press Enter. |
I've tried to implement this yesterday, but didn't get it to do anything besides MBS trying to report an error to sentry 🤯 Without my copies of the MBS type definitions it would have been a pain to find out which properties are expected by the exposed methods. It expects the properties of the existing relationship to be passed three times currently... maybe I should ask bitmap if this is really necessary. |
Hi @kellnerd, |
There is only the bookmarklet version so far as I haven't thought about the UI to expose it yet. |
How do you know when this happens? |
I see the sentry requests in the browser console. But uBlock Origin blocks sentry requests by default, so I don't spam MBS with confusing errors while developing. Only sometimes when I have no idea why one of my scripts is not working, I allow these sentry requests to go through. This seems to be necessary to be able to inspect their body with the error details in the browser console. |
The selector for the checkboxes has changed and
MB.relationshipEditor.UI.checkedWorks()
no longer exists. I am trying to implement this feature with the new exposed functions.The text was updated successfully, but these errors were encountered: