-
-
Notifications
You must be signed in to change notification settings - Fork 204
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
docs: Update README
#471
docs: Update README
#471
Conversation
|
That's confusing 😕 Any misalignment here? Now I feel like I was doing something wrong and wasted time for nothing as of the incorrect(?) pointer from this repo dev 😞 |
I could've reverted the link change (still the link in current README points to an outdated GAS project) since the other part of PR (and the attached patch) does add value I guess. Ain't it? |
@jonas0b1011001 - let me know what your input is here. If this doc link is owned by you (and you want to move the ownership that way so I don't have to be involved to update it) then I'm fine with that. But this link needs to be publicly accessible. @yermulnik Yes, the link intentionally points to an outdated GAS project. You can think of that link as "the latest release" and the code in our repo as "the development branch". We could track this discrepancy through different branches on GitHub (eg "release" and "dev" or whatever) but we haven't done that. We update https://github.com/derekantrican/GAS-ICS-Sync/releases whenever we update that script file (and you can see that was last done around a year ago) |
@derekantrican Got it, thanks for explanation. Maybe it would make sense to update README/Wiki with details on release vs development versions as e.g. for me it appeared to be a bit of a fuss to figure out what was going wrong as I wanted to use most recent version and simply missed that |
Sorry for the late response.
That link indeed is pointing to a script on my google drive and is used by me to allow people in the issues/discussion section to quickly test if the new build is solving their problems. As i was trying to express here, the link is not intended to replace the 'stable' link in the readme.
I was not aware that the only missing piece in the wiki was bottom line:
Thoughts? |
I obviously have insufficient permissions to PR to Wiki, hence here's a relevant patch:
0001-docs-Add-filters.gs-to-Wiki.patch.txt
Fixes #470
Refs: