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

[WIP] Add item uid #1017

Merged
merged 7 commits into from
Feb 3, 2019
Merged

[WIP] Add item uid #1017

merged 7 commits into from
Feb 3, 2019

Conversation

logmanoriginal
Copy link
Contributor

@logmanoriginal logmanoriginal commented Jan 22, 2019

This PR adds the parameter "uid" to $item.

References #977, #1005

Todo

  • Testing
  • Documentation

'uid' represents the unique id for a feed item. This item is null by
default and can be set to any string value. The provided string value
is always hashed to sha1 to make it the same length in all cases.

References #977, #1005
@logmanoriginal logmanoriginal added the New-Feature This is a new feature label Jan 22, 2019
@logmanoriginal logmanoriginal self-assigned this Jan 22, 2019
@fulmeek
Copy link
Contributor

fulmeek commented Jan 23, 2019

Nice, looks good :)

I've added a PR to your PR for some suggestions:
https://github.com/LogMANOriginal/rss-bridge/pull/1

I think it should work well then.

@logmanoriginal
Copy link
Contributor Author

Thanks for the PR, it seems to work well now. 👍

@logmanoriginal logmanoriginal merged commit 394149b into RSS-Bridge:master Feb 3, 2019
@logmanoriginal logmanoriginal deleted the ItemUid branch June 23, 2019 07:38
infominer33 pushed a commit to web-work-tools/rss-bridge that referenced this pull request Apr 17, 2020
'uid' represents the unique id for a feed item. This item is null by
default and can be set to any string value. The provided string value
is always hashed to sha1 to make it the same length in all cases.

References RSS-Bridge#977, RSS-Bridge#1005
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New-Feature This is a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants