minds-ai / zoom-drive-connector

Automatically uploads Zoom meeting recordings to Google Drive.
Other
34 stars 7 forks source link

Repository Clean Up and Updates #5

Closed MrFlynn closed 6 years ago

MrFlynn commented 6 years ago

After my discussion with Tijmen last week (9/5), I would like to nominate this project for potential open sourcing sometime in the future (pending legal approval and further discussion). Regardless of that potential project, I think it would be beneficial to fix a couple of things in this repository.

Initial Task List:

MrFlynn commented 6 years ago

That's basically what I was looking to do with this repository. I did the same thing in another project, though I also have the portable environment file there.

MrFlynn commented 6 years ago

Referencing issue #7 in this general improvement thread.

MrFlynn commented 6 years ago

For future open sourcing, a better README describing the install/run steps will also be required.=

MrFlynn commented 6 years ago
MrFlynn commented 6 years ago
MrFlynn commented 6 years ago

Make sure to remove hardcoded IDs from previous commits when completing the first item.

Commands required to remove files and strings:

$ bfg --delete-files some_file zoom-drive-connector.git
$ bfg --replace-text text zoom-drive-connector.git
MrFlynn commented 6 years ago

Turns out removing strings and removing files from a Git repository hosted on GitHub is impossible because GitHub makes refs to pull requests read only and we cannot delete the PRs or refs ourselves.

I'm closing this issue because there doesn't seem like a reasonable solution to this other than asking GitHub to remove pull requests (which I would rather we not do).