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

Fix #5: Initial commit of html, css, and images #9

Conversation

ragnarokatz
Copy link
Contributor

Initial commit of html, css, and images copied over from current planet site hosted on zenit.

@ragnarokatz
Copy link
Contributor Author

reference issue #5

@ragnarokatz
Copy link
Contributor Author

ragnarokatz commented Oct 30, 2019

how it currently looks: here

@Reza-Rajabi
Copy link
Contributor

Now that we are using a static web page rather than a dynamic component-based, don't we need to use some template like handlebars to load the data on the compiled html instead of repeating html tags for each feed?

@ODAVING
Copy link
Contributor

ODAVING commented Nov 5, 2019

I’m perfectly dine with merging this and continuously working on it! And developing it further! @humphd would that be feasible?

@Reza-Rajabi Reza-Rajabi added this to Done in Main Nov 6, 2019
@Reza-Rajabi Reza-Rajabi moved this from Done to Review in progress in Main Nov 6, 2019
@cindyledev
Copy link
Contributor

cindyledev commented Nov 7, 2019

This is good but it's currently over 20 commits behind master, please rebase this.

@cindyledev cindyledev mentioned this pull request Nov 7, 2019
@ragnarokatz
Copy link
Contributor Author

I can get this updated. I'll send out a review request once it's done.

@ragnarokatz ragnarokatz force-pushed the issue-4-copy-basic-template-css-files-over branch from 748f08e to 43d95fa Compare November 7, 2019 17:30
@ragnarokatz ragnarokatz changed the title Initial commit of html, css, and images Fix #5: Initial commit of html, css, and images Nov 7, 2019
@ODAVING
Copy link
Contributor

ODAVING commented Nov 7, 2019

@ragnarokatz can you please squash both commits together?

on zenit

Initial commit of html, css, and images copied over from current planet on zenit

Replace old doctype html with simpler one
Copy link
Contributor

@lucacataldo lucacataldo left a comment

Choose a reason for hiding this comment

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

Looks good to me as a starting point, commits should be squashed though

@ragnarokatz ragnarokatz force-pushed the issue-4-copy-basic-template-css-files-over branch from 43d95fa to 856f0ae Compare November 7, 2019 17:41
@ragnarokatz
Copy link
Contributor Author

ragnarokatz commented Nov 7, 2019

@ODAVING @lucacataldo done. please take another look when you have time, thanks.

Copy link
Contributor

@ODAVING ODAVING left a comment

Choose a reason for hiding this comment

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

Nicely done! thank you

Copy link
Contributor

@lucacataldo lucacataldo left a comment

Choose a reason for hiding this comment

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

👍

@ragnarokatz ragnarokatz merged commit a18e3fd into Seneca-CDOT:master Nov 7, 2019
Main automation moved this from In progress/Review to Done Nov 7, 2019
@ragnarokatz ragnarokatz deleted the issue-4-copy-basic-template-css-files-over branch November 7, 2019 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Main
Done
Development

Successfully merging this pull request may close these issues.

None yet

5 participants