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.

Sign up for GitHub

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

Jump to bottom

Community PR fixes #72

Merged
merged 5 commits into from
Feb 7, 2024
Merged

Community PR fixes #72

merged 5 commits into from
Feb 7, 2024

Conversation

durch
Copy link
Contributor

@durch durch commented Feb 6, 2024 •
edited
Loading

Requirements obsidianmd/obsidian-releases#2938:

  • version
  • description
  • NodeJS.Timeout -> window.setTimeout
  • fetch -> requestUrl
  • Flow -> lowercase
  • Settings headings
  • readFileSync -> Vault.readBinary
  • replace randomUUID

@durch durch requested a review from muness February 6, 2024 14:07
@muness muness merged commit e74bb29 into master Feb 7, 2024
2 checks passed
@muness muness deleted the community-pr-fixes branch February 7, 2024 11:24
@muness muness restored the community-pr-fixes branch February 7, 2024 11:25
This was referenced Feb 7, 2024
Merged
Merged
@muness
Copy link
Member

muness commented Feb 7, 2024 •
edited
Loading

I got click happy in the UI. Continued in #76

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@muness muness muness approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@durch @muness