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

Mobile #66

Closed
wants to merge 34 commits into from
Closed

Mobile #66

wants to merge 34 commits into from

Conversation

Vinzent03
Copy link
Owner

@Vinzent03 Vinzent03 commented Apr 15, 2021

Checklist

  • commit
  • push
  • pull
  • status
  • error handling
  • validation check
  • Repository init
  • settings pane with configurations
  • disable status bar on mobile
  • port simple-git to new plugin structure
  • documentation
  • community security check (I want to check the secure use of the personal access-token)

@Vinzent03 Vinzent03 added the enhancement New feature or request label Apr 15, 2021
@Vinzent03 Vinzent03 self-assigned this Apr 15, 2021
@Vinzent03 Vinzent03 mentioned this pull request Apr 20, 2021
@zanets zanets mentioned this pull request May 2, 2021
@hardenchant
Copy link

I want install plugin version on my phone from this branch for tests, how can i do this?
Build and replace files main.js and manifest.json at .obsidian folder on my android smartphone not working for me.

@Vinzent03
Copy link
Owner Author

@hardenchant You need to remove any node imports. So you have to delete the simple git file. Just ignore the import errors now. I think there is also a child process import in main.ts. this import needs to be deleted too. A bit tricky I know. The only import/require at the top of main.js should be obsidian.

@Vinzent03 Vinzent03 force-pushed the master branch 2 times, most recently from 8a58556 to 8d64c95 Compare August 12, 2021 16:45
@Ellpeck
Copy link

Ellpeck commented Feb 11, 2022

Is there anything we can do to help with this? I am looking forward to this being implemented; this plugin is awesome!!

@Vinzent03
Copy link
Owner Author

See merged #270

@Vinzent03 Vinzent03 closed this Aug 14, 2022
@Vinzent03 Vinzent03 deleted the mobile branch January 7, 2023 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants