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

Is upcoming broken? #88

Open
egelmex opened this issue Jan 11, 2015 · 7 comments
Open

Is upcoming broken? #88

egelmex opened this issue Jan 11, 2015 · 7 comments
Assignees

Comments

@egelmex
Copy link
Owner

egelmex commented Jan 11, 2015

No description provided.

@graymalkin
Copy link
Collaborator

It would appear there's been some sort of API change.

      if (this.opts.feed_url) {
        return this.opts.feed_url;
      } else if (this.opts.mode === 'updates') {
        return "https://www.google.com/calendar/feeds/" + this.opts.calid + "/public/full?alt=json-in-script&max-results=" + this.opts.maxitem + "&orderby=lastmodified&sortorder=descending";
      } else {
        return "https://www.google.com/calendar/feeds/" + this.opts.calid + "/public/full?alt=json-in-script&max-results=" + this.opts.maxitem + "&orderby=starttime&futureevents=true&sortorder=ascending&singleevents=true";
      }

See jquery.gcal_flow.js:125

Investigating, it looks like, the URL has changed slightly.

This does work (in the potential patch):
https://www.google.com/calendar/feeds/tinkersoc.org_jgvajjns79sp34mvqntl81rub0%40group.calendar.google.com/public/basic?alt=json-in-script

This does not work (in the current implementation):
https://www.google.com/calendar/feeds/tinkersoc.org_jgvajjns79sp34mvqntl81rub0%40group.calendar.google.com/public/full?alt=json-in-script

The change is from basic to full at the end of the URL.

@graymalkin
Copy link
Collaborator

Potentially it's sane to have a submodule to the module's source, rather than a copy?
https://github.com/sugi/jquery-gcal-flow/

@egelmex
Copy link
Owner Author

egelmex commented Jan 15, 2015

Last time I looked submodules in git didn't act like you would expect, but
I hear there has been some work done on them.

If you want to experiment and bring up a pull req I will happily merge. =
On Thu Jan 15 2015 at 10:05:52 AM Simon Cooksey [email protected]
wrote:

Potentially it's sane to have a submodule to the module's source, rather
than a copy?
https://github.com/sugi/jquery-gcal-flow/


Reply to this email directly or view it on GitHub
#88 (comment).

@graymalkin
Copy link
Collaborator

Submodules are essentially "I want this git repo here, at this commit pointer" -- you can do a reccursive git pull, to update all submodules.

I'll have a play when I'm not at work, and have an internet connection at home that allows me to ssh >.<

@egelmex
Copy link
Owner Author

egelmex commented Jan 27, 2015

Any change you will have time to make a pull req to get this fixed (if if hacky and raise a new ticket to improve it?)

@graymalkin
Copy link
Collaborator

Should be easy enough, the only difficulty will be forcing jekyll to compile the coffee script without adding front matter to the file (as that would defy the point of having a git submodule)

It may be that I add a Makefile which does some legwork first. That could be very nice for doing things like

make serve
make local
make deply

Etc. Thoughts?

@egelmex
Copy link
Owner Author

egelmex commented Jan 27, 2015

There is already a make script on my dev server (you should have access)
that I use to push to S3. Might be worth looking at/extending that.

On Tue Jan 27 2015 at 12:05:45 PM Simon Cooksey [email protected]
wrote:

Should be easy enough, the only difficulty will be forcing jekyll to
compile the coffee script without adding front matter to the file (as that
would defy the point of having a git submodule)

It may be that I add a Makefile which does some legwork first. That could
be very nice for doing things like

make serve
make local
make deply

Etc. Thoughts?


Reply to this email directly or view it on GitHub
#88 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants