Skip to content

Ical link - #116

Merged
JJ-8 merged 4 commits into
mainfrom
ical-link
Apr 28, 2024
Merged

JJ-8 merged 4 commits into
mainfrom
ical-link

Conversation

@B-i-t-K

@B-i-t-K B-i-t-K commented Nov 30, 2021

Copy link
Copy Markdown
Member

An ical link is now available on /calendar.ics.

It contains all the ctfs with their description and dates.
This endpoint is protected with a token by default,
the password can be changed/removed by the admin.

BitK added 3 commits November 30, 2021 02:28
An ical link is now available on /calendar.ics.
It contains all the ctfs with their description and dates.
This endpoint is protected with a token by default,
the password can be changed/removed by the admin.
@B-i-t-K

B-i-t-K commented Nov 30, 2021

Copy link
Copy Markdown
Member Author

Calendar:
2021-11-30_03-04

Admin:
image

gen_random_bytes is more suited for crypto purpose

@XeR XeR left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not test this yet. These are only questions

Comment thread api/src/index.ts
})
);
app.use(postgraphile(getDbUrl("user"), "ctfnote", postgraphileOptions));
app.use("/calendar.ics", icalRoute(pool));

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see the pool is only used for the ical route.
Can we use this for postgraphile too ? It would be nice to share the same connection.

Comment thread front/.gitignore
*.sln
.vscode
graphql.schema.json

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since both graphql.schema.json and schema.ts are deleted, shouldn't we add both to the gitignore ?

@frereit frereit mentioned this pull request Feb 7, 2022
@JJ-8
JJ-8 merged commit b97d1f6 into main Apr 28, 2024
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

Successfully merging this pull request may close these issues.

3 participants