Skip to content

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

Open
PaarthAgarwal wants to merge 2 commits intoaccordproject:mainfrom
PaarthAgarwal:PaarthAgarwal/i404/copyrightyear-update
Open

template-library(build/htmls): update footer - I404#405
PaarthAgarwal wants to merge 2 commits intoaccordproject:mainfrom
PaarthAgarwal:PaarthAgarwal/i404/copyrightyear-update

Conversation

@PaarthAgarwal
Copy link
Copy Markdown

Updated copyright year

Changes

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

Screenshots or Video

image

Related Issues

PaarthAgarwal and others added 2 commits February 1, 2022 18:22
@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
Copy Markdown
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
Copy Markdown
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