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

Chainlink fix #666

Closed
wants to merge 5 commits into from
Closed

Chainlink fix #666

wants to merge 5 commits into from

Conversation

jvorob
Copy link
Contributor

@jvorob jvorob commented Sep 8, 2023

Rewrote operation logic and commented
Changed disableEditing to use srcRowId=new or srcRowId=null

Rewrote operation logic and commented
Changed disableEditing to use srcRowId=new or srcRowId=null
Weird behavior: filterlinking calls makeQuery twice in quick
succession, but for some reason the calls resolve out of order
when starting from a refresh. However, after moving the cursor around
a bit, they START BEHAVING CORRECTLY AGAIN???
@jvorob jvorob added the preview Launch preview deployment of this PR label Sep 12, 2023
@github-actions
Copy link
Contributor

Deployed as https://grist-chainlink-fix.fly.dev (until 2023-10-12T23:01:41.624Z)

@berhalak
Copy link
Contributor

Closing for now, stored link to this in the internal document about linking improvement.

@berhalak berhalak closed this Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preview Launch preview deployment of this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants