CCCHH/ccchh-website
CCCHH/ccchh-website
Watch
3
3
Fork
You've already forked ccchh-website
 4
Code Issues 5 Pull requests 8 Activity Actions

Improve RSS Feed #24

Merged
echtnurich merged 1 commit from rss-improvements into main 2024-01-27 23:15:23 +01:00
Conversation 5 Commits 1 Files changed 1 +5
echtnurich commented 2023-12-13 18:56:33 +01:00 (Migrated from gitlab.hamburg.ccc.de)
Copy link

Todo/Ideas:

  • Add author field to rss articles
  • Add rss icon to site?
Todo/Ideas: - [x] Add author field to rss articles - [ ] ~~Add rss icon to site?~~
echtnurich commented 2024-01-23 18:14:10 +01:00
Owner
Copy link

miau?

miau?
jtbx force-pushed rss-improvements from 72ae6166f1 to 19497d904f
All checks were successful
/ build (pull_request) Successful in 38s
Details
2024-01-23 20:46:26 +01:00 Compare
jtbx requested changes 2024-01-23 20:56:16 +01:00
jtbx left a comment
Copy link

By the spec, the author field shall be an email address. So this is not compatible with the spec.

If we misuse the field as free text, we should add a space after each comma.

By the [spec](https://cyber.harvard.edu/rss/rss.html#ltauthorgtSubelementOfLtitemgt), the author field shall be an email address. So this is not compatible with the spec. If we misuse the field as free text, we should add a space after each comma.
jtbx force-pushed rss-improvements from 19497d904f
All checks were successful
/ build (pull_request) Successful in 38s
Details
to 1b8196040e
All checks were successful
/ build (pull_request) Successful in 36s
Details
2024-01-27 23:13:20 +01:00 Compare
jtbx commented 2024-01-27 23:13:39 +01:00
Copy link

Author list has spaces after commas now.

Author list has spaces after commas now.
jtbx approved these changes 2024-01-27 23:14:41 +01:00
c6ristian approved these changes 2024-01-27 23:14:54 +01:00
jtbx merged commit 6911482c0e into main 2024-01-27 23:15:23 +01:00
jtbx deleted branch rss-improvements 2024-01-27 23:15:23 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
jtbx
c6ristian
Labels
Clear labels   
bug
Something is not working

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
help wanted
Need some help

  
invalid
Something is wrong

  
question
More information is needed

  
wontfix
This won't be fixed

No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
CCCHH/ccchh-website!24
No description provided.