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

template-library(build/htmls): update footer - I404 #405

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

PaarthAgarwal
Copy link

Updated copyright year

Changes

changed 'Copyright 2018-2019' to ' Copyright 2018-2022'

Screenshots or Video

image

Related Issues

@PaarthAgarwal PaarthAgarwal changed the title template library(build/htmls) update footer - I404 template-library(build/htmls): update footer - I404 Feb 1, 2022
@jeromesimeon jeromesimeon self-requested a review February 4, 2022 18:34
Copy link
Member

@jeromesimeon jeromesimeon left a comment

Choose a reason for hiding this comment

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

hi @PaarthAgarwal thanks for your PR. Two main comments:

  1. you are changing files that are generated during the build process, instead you should change the source templates for those pages
  2. I also noticed changes to some of the .cta archives, are those just erroneous updates?

@PaarthAgarwal
Copy link
Author

hi @PaarthAgarwal thanks for your PR. Two main comments:

  1. you are changing files that are generated during the build process, instead you should change the source templates for those pages
  2. I also noticed changes to some of the .cta archives, are those just erroneous updates?
  1. I understand now. The .njk files in the views folder are the source templates, right?
  2. Yeah it was erroneous.
    If I got the first point right may I proceed and modify it?

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