-
-
Notifications
You must be signed in to change notification settings - Fork 6
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] Breaking: Improve the new Authors API #1798
Conversation
caendesilva
commented
Jul 7, 2024
•
edited
Loading
edited
- Fixes Support setting authors in the YAML configuration #1719
- Fixes Improve the author feature to allow more schema fields #1778
- Fixes [2.x] Find a better way to reduce author username duplication #1797
f4ec5a5
to
27386ba
Compare
Updates to use the new format specified in #1782
Normalizes the API
27386ba
to
95d01b7
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 2.x-dev hydephp/develop#1798 +/- ##
===========================================
Coverage 100.00% 100.00%
- Complexity 1838 1842 +4
===========================================
Files 188 188
Lines 4862 4877 +15
===========================================
+ Hits 4862 4877 +15 ☔ View full report in Codecov by Sentry. |
Enforces the new state. Consider adding a BadConfigurationException?
This leads to duplication which gets confusing
5e000ad
to
fbfa507
Compare
If we can't find the line, it's probably because an internal/package developer made a mistake
f9a9ebe
to
81fc538
Compare
81fc538
to
ba2a352
Compare
eda0c62
to
50de689
Compare
4f6a195
to
f317b2d
Compare
ba1c18b
to
eebecc9
Compare
eebecc9
to
5fda706
Compare
|
||
// The following settings are not used in the bundled templates, | ||
// but you can use them in your own custom views, for example. | ||
// bio: 'The mysterious author of HydePHP', | ||
// avatar: 'avatar.png', | ||
// socials: [ | ||
// 'twitter' => 'HydeFramework', | ||
// 'github' => 'hydephp', | ||
// ], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we do not need this here, as they are not yet used, and they are in the documentation. However, if https://github.com/hydephp/develop/issues/1762 will use these they can be kept, hence why I'm holding off on removing them.
[2.x] Breaking: Improve the new Authors API
[2.x] Breaking: Improve the new Authors API