Skip to content

✨ Update footer text#2

Open
Pinimes wants to merge 1 commit into
aweandreverence:masterfrom
Pinimes:features/update-footer-text
Open

✨ Update footer text#2
Pinimes wants to merge 1 commit into
aweandreverence:masterfrom
Pinimes:features/update-footer-text

Conversation

@Pinimes

@Pinimes Pinimes commented Oct 26, 2023

Copy link
Copy Markdown
Contributor

Status

Ready

Description

  • changed the footer from ab506.church to Awe and Reverence Inc

Screenshot

image

@Pinimes Pinimes self-assigned this Oct 26, 2023
@Pinimes Pinimes requested a review from a team as a code owner October 26, 2023 09:04
Comment thread components/footer.js
</div>
<div className={css.copyright}>
<p>&copy; ab506.church {CURRENT_YEAR}</p>
<p>&copy; Awe and Reverence {CURRENT_YEAR}</p>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This should be a constant. You can put the constant in constants/seo.py

Names can be consider for constant:

  • PARENT_SITE_NAME
  • GROUP_SITE_NAME
  • AR_SITE_NAME

Maybe it is best that @jontsai decide the name.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hardcoded is fine for now.

Realistically, we may want to create a 3rd GitHub repository as a submodule to store the constants.

Or, we might create a public NPMJS package hosted in our GitHub repository (such as aweandreverence-nextjs-common. @goztrk thoughts?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ok, done

@jontsai jontsai left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Comment thread components/footer.js
</div>
<div className={css.copyright}>
<p>&copy; ab506.church {CURRENT_YEAR}</p>
<p>&copy; Awe and Reverence {CURRENT_YEAR}</p>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hardcoded is fine for now.

Realistically, we may want to create a 3rd GitHub repository as a submodule to store the constants.

Or, we might create a public NPMJS package hosted in our GitHub repository (such as aweandreverence-nextjs-common. @goztrk thoughts?

@Pinimes Pinimes force-pushed the features/update-footer-text branch from cd5af9d to 42e6a9f Compare October 26, 2023 15:14

@lleung030 lleung030 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

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.

4 participants