[@astrojs/rss] Quality-of-Life Improvement to items property-related error
#9299
Conversation
… feed when using collections. Specifically, it provides more context to the error message thrown when the object passed to the `items` property is missing any of the three required keys or if one of those keys is mistyped.
🦋 Changeset detectedLatest commit: 2b3a471 The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Just to put it out there, I'm not married to the copy language used, nor the method. I was a little confused by the use of |
There was a problem hiding this comment.
A docs review only: the proposed error message is fantastic. Thank you for the care and attention to detail @cdvillard! Massaged the changeset a little, so see what you think about that.
@sarah11918 suggested a change to the verbiage that properly formatted the update detail in question. Accepting the suggestion. Co-authored-by: Sarah Rainsberger <[email protected]>
Changes
This commit addresses a quality-of-life concern when setting up a RSS feed using collections. Specifically:
itemsproperty is missing any of the three required keys or if one of those keys is mistyped.Testing
I tested this change similarly to how I encountered #9171. Using the
examples/blogproject, I simply messed around with the properties ofitemsto ensure an error would render, and that my changes would build and reflect appropriately.Before:
After:
Docs
This shouldn't necessitate any changes to the docs as this way entirely done to help improve the error solely. Usage of the package remains unchanged.