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

Bug: Fixed footer #175

Closed
wants to merge 1 commit into from
Closed

Conversation

Meetjain1
Copy link

Related Issue

[Cite any related issue(s) this pull request addresses. If none, simply state "None”]
None

Description

PR Title: Update Footer to Include Dynamic Copyright Year

Description:

This PR enhances the footer section of the website by adding the current copyright year, which is fetched dynamically. This ensures that the copyright year will always be up-to-date without requiring manual updates each year.

Changes:

  1. Footer HTML: Updated the footer to include a placeholder for the copyright year.
  2. JavaScript: Added a script to fetch the current year dynamically and insert it into the footer.
  3. CSS: Ensured that the styling of the footer remains consistent with the new dynamic content.

Benefits:

  • Automation: The copyright year updates automatically, reducing maintenance overhead.
  • Accuracy: Ensures the displayed year is always current, improving the website's professionalism.

Type of PR

  • Bug fix
  • Feature enhancement
  • Documentation update
  • Other (specify): _______________

Screenshots / videos (if applicable)

[Attach any relevant screenshots or videos demonstrating the changes]

Checklist:

  • I have performed a self-review of my code
  • I have read and followed the Contribution Guidelines.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have commented my code, particularly in hard-to-understand areas.

Additional context:

[Include any additional information or context that might be helpful for reviewers.]

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Awesome work, @Meetjain1! 🎉 Thank you for your contribution and efforts to enhance NEWS-AGGREGATOR-PROJECT ! We will review your changes soon and provide feedback.

@Meetjain1
Copy link
Author

@SiddharthBahuguna please review this pr

@Meetjain1
Copy link
Author

@SiddharthBahuguna please review this pr

@SiddharthBahuguna

@SiddharthBahuguna
Copy link
Owner

@Meetjain1 this PR does not add value in the project.

@Meetjain1
Copy link
Author

@Meetjain1 this PR does not add value in the project.

why so ?

@Meetjain1
Copy link
Author

@Meetjain1 this PR does not add value in the project.

why so @SiddharthBahuguna ?

@Meetjain1
Copy link
Author

@SiddharthBahuguna

@Meetjain1 this PR does not add value in the project.

why so @SiddharthBahuguna ?

please review this pr . before gssoc ends . Thanks

@SiddharthBahuguna
Copy link
Owner

SiddharthBahuguna commented Aug 9, 2024 via email

@SiddharthBahuguna
Copy link
Owner

@Meetjain1 you have just edited 1 line which was also not required.

This task was also not assigned to you by any of the mentors.

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.

2 participants