Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Tag class, Tags options and IFrontMatterWithTags. #52

Merged
tesar-tech merged 7 commits into
masterfrom
rework-services
Jan 5, 2025
Merged

Tag class, Tags options and IFrontMatterWithTags. #52
tesar-tech merged 7 commits into
masterfrom
rework-services

Conversation

@tesar-tech

@tesar-tech tesar-tech commented Nov 6, 2024

Copy link
Copy Markdown
Collaborator

The initial idea was shifted significantly.

Now the pain points has been solved much smoother. The tags work as expected.

@netlify

netlify Bot commented Nov 6, 2024

Copy link
Copy Markdown

Deploy Preview for blazorstatic ready!

Name Link
🔨 Latest commit e3be540
🔍 Latest deploy log https://app.netlify.com/sites/blazorstatic/deploys/677a81f925e6990009ac860a
😎 Deploy Preview https://deploy-preview-52--blazorstatic.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@junovell

junovell commented Nov 6, 2024

Copy link
Copy Markdown
Contributor

for tags why not use attributes? we already use reflection to get the site's routes so it shouldn't be an issue

i'm thinking something like this:

class MyFrontMatter {

    public string Title { get; set; }
    [BlogTags]
    public string[] categories { get; set; }
}

that way the user can name the variable whatever they wish or omit it completely if they don't need it without us having to maintain 2 different interfaces

@tesar-tech

Copy link
Copy Markdown
Collaborator Author

for tags why not use attributes?

That made me think about the Tags from a different perspective...

Because Tags wont be List<string> anymore, but we still need the List<string> to parse the data from yml...

Which means we need to "free" the IFrontMatter from anything that is not used as strongly typed metadata... Which means we need to put it in Post (List<string> Tags in FrontMatter and List<Tag> Tags in Post), which kind of means we need to provide an interface for the Post itself...

It is somehow getting together. I will see, how the Attribute goes along with it..

@tesar-tech tesar-tech changed the title Change for services and options and Blog related stuff in own namespace. Tag class, Tags options and IFrontMatterWithTags. Nov 10, 2024
@tesar-tech

Copy link
Copy Markdown
Collaborator Author

The BlazorStaticWebsite/Content/Blog/release-1.0.0-beta.14.md file sums up the changes

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jan 5, 2025

Copy link
Copy Markdown

Deploying blazorstatic with  Cloudflare Pages  Cloudflare Pages

Latest commit: e3be540
Status: ✅  Deploy successful!
Preview URL: https://bf938d20.blazorstatic.pages.dev
Branch Preview URL: https://rework-services.blazorstatic.pages.dev

View logs

@tesar-tech tesar-tech merged commit 96c79e0 into master Jan 5, 2025
@tesar-tech tesar-tech deleted the rework-services branch January 5, 2025 13:00
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