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

[2.x] Overhaul the blog post authors feature #1782

Merged
merged 93 commits into from
Jul 7, 2024

Conversation

caendesilva
Copy link
Member

@caendesilva caendesilva commented Jul 5, 2024

Abstract

This pull request makes major improvements into the usability and design of the blog post author feature.

See the 1.x tie in at #1794. See also #1798

Roadmap

Changes

  • Breaking: The hyde.authors config setting should now be keyed by the usernames
  • Removed: The deprecated PostAuthor::getName() method is now removed (use $author->name)
  • Feature: We now support setting authors in the YAML configuration! Fixes #1719
  • Feature: Added a $author->getPosts() method to get all author's posts
  • Feature: Authors now can also have custom biographies and social media links
  • The PostAuthor class is now Arrayable and JsonSerializable
  • The collection of site authors are now stored in the HydeKernel
  • Authors can additionally be accessed through Hyde::authors()

Copy link

codecov bot commented Jul 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (ebe55fb) to head (8f1134d).

Additional details and impacted files
@@             Coverage Diff             @@
##             2.x-dev     #1782   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity      1823      1832    +9     
===========================================
  Files            186       187    +1     
  Lines           4824      4849   +25     
===========================================
+ Hits            4824      4849   +25     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@caendesilva caendesilva force-pushed the overhaul-the-authors-feature branch from b25b63a to f02091d Compare July 5, 2024 17:16
@caendesilva caendesilva force-pushed the overhaul-the-authors-feature branch from 14f22a8 to 86b5de9 Compare July 6, 2024 16:25
@caendesilva caendesilva force-pushed the overhaul-the-authors-feature branch from c0187c4 to 62582e4 Compare July 6, 2024 18:00
@caendesilva caendesilva force-pushed the overhaul-the-authors-feature branch from f365d0e to 5fb193e Compare July 6, 2024 18:05
caendesilva added a commit that referenced this pull request Jul 6, 2024
caendesilva added a commit that referenced this pull request Jul 6, 2024
caendesilva added a commit that referenced this pull request Jul 6, 2024
caendesilva added a commit that referenced this pull request Jul 6, 2024
@caendesilva caendesilva merged commit 25dd9da into 2.x-dev Jul 7, 2024
8 checks passed
@caendesilva caendesilva deleted the overhaul-the-authors-feature branch July 7, 2024 09:22
caendesilva added a commit that referenced this pull request Jul 7, 2024
Updates to use the new format specified in #1782
@caendesilva caendesilva mentioned this pull request Jul 9, 2024
74 tasks
@caendesilva caendesilva added this to the v2 milestone Jul 9, 2024
@caendesilva caendesilva mentioned this pull request Jul 10, 2024
4 tasks
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.

1 participant