Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

add microsite calendar #890

Closed
wants to merge 1 commit into from
Closed

add microsite calendar #890

wants to merge 1 commit into from

Conversation

@Yogendra0Sharma
Copy link

@Yogendra0Sharma Yogendra0Sharma commented Aug 20, 2020

Related Issues

#883

@benhalverson
Copy link
Member

benhalverson commented Aug 20, 2020

/preview

@benhalverson benhalverson added the create-preview Generate preview on staging.nodejs.dev label Aug 20, 2020
@github-actions github-actions bot removed the create-preview Generate preview on staging.nodejs.dev label Aug 20, 2020
@github-actions
Copy link

github-actions bot commented Aug 20, 2020

Please find a preview at: https://staging.nodejs.dev/890/

designMoreWeb
designMoreWeb reviewed Aug 20, 2020
View reviewed changes
Copy link
Contributor

@designMoreWeb designMoreWeb left a comment

Choose a reason for hiding this comment

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

I am not able to see the calendar the preview takes me to the main homepage of the site

@mikeesto
Copy link
Member

mikeesto commented Aug 22, 2020

I am not able to see the calendar the preview takes me to the main homepage of the site

I think the direct link is https://staging.nodejs.dev/890/sites/calendar/

designMoreWeb
designMoreWeb approved these changes Aug 22, 2020
View reviewed changes
Copy link
Contributor

@designMoreWeb designMoreWeb left a comment

Choose a reason for hiding this comment

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

LGTM

benhalverson
benhalverson reviewed Aug 22, 2020
View reviewed changes
benhalverson
benhalverson suggested changes Aug 22, 2020
View reviewed changes
Copy link
Member

@benhalverson benhalverson left a comment

Choose a reason for hiding this comment

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

Can you make the calendar work with dark mode?
Here are two ways.
https://stackoverflow.com/questions/217776/how-to-apply-css-to-iframe or using the gCal API
https://developers.google.com/calendar/v3/reference/colors

Also please address the conflicts

@nschonni nschonni mentioned this pull request Aug 28, 2020
Open
@benhalverson
Copy link
Member

benhalverson commented Oct 4, 2020

@Yogendra0Sharma Are you able to fix the conflicts and address my other comments about darkmode?

@benhalverson benhalverson linked an issue Oct 15, 2020 that may be closed by this pull request
microsite calendar #883
Closed
10 tasks
@benhalverson
Copy link
Member

benhalverson commented Mar 14, 2021

Closing as abandoned.

@benhalverson benhalverson mentioned this pull request Sep 14, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers

@benhalverson benhalverson benhalverson requested changes

@designMoreWeb designMoreWeb designMoreWeb approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

microsite calendar

4 participants

@Yogendra0Sharma @benhalverson @mikeesto @designMoreWeb