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

Problem adding new feeds #6

Closed
mzfr opened this issue Feb 17, 2023 · 6 comments
Closed

Problem adding new feeds #6

mzfr opened this issue Feb 17, 2023 · 6 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@mzfr
Copy link

mzfr commented Feb 17, 2023

I am trying to add new feeds and getting the same error for every feed that I've tried so far:

 JOURNALIST_API_USERNAME=admin JOURNALIST_API_PASSWORD=admin \
  ./redacteur add feed
URL: https://ahmet.im/blog/feed/rss.xml
Name: /home/alp
Group: dev
{"success":false,"feed":null,"message":"pq: ON CONFLICT DO UPDATE requires inference specification or constraint name"}%
@LeKovr
Copy link
Contributor

LeKovr commented Mar 27, 2023

It seems this may help

@mrusme mrusme added bug Something isn't working help wanted Extra attention is needed labels Mar 27, 2023
LeKovr added a commit to LeKovr/journalist that referenced this issue Mar 28, 2023
LeKovr added a commit to LeKovr/journalist that referenced this issue Mar 28, 2023
@LeKovr
Copy link
Contributor

LeKovr commented Mar 28, 2023

2 places fixed, but I'm not so familiar with ent and might be there is something more elegant than OnConflictColumns("url", "username", "password").
#7

mrusme added a commit that referenced this issue Mar 29, 2023
@mrusme
Copy link
Owner

mrusme commented Mar 29, 2023

@LeKovr thank you very much! @mzfr please test using master!

@mzfr
Copy link
Author

mzfr commented Mar 30, 2023

@mrusme Sorry but I have purged the journalist setup and started using yarr(https://github.com/mzfr/yarr). So I don't have a way to test this now.

If you and @LeKovr think the bug is squashed then feel free to close this ticket

@LeKovr
Copy link
Contributor

LeKovr commented Mar 30, 2023

It could be 2 other places:

$ grep -rin "OnConflict()" *
web/actions/read.go:129:		OnConflict().
web/actions/read_all.go:70:		OnConflict().

But I don't know how to test them yet

@mrusme
Copy link
Owner

mrusme commented Jul 19, 2024

Using the latest master:

▲ JOURNALIST_API_USERNAME=admin JOURNALIST_API_PASSWORD=admin \               ⇡ master 57875b rad:z2qiLZSSYewxT8Y1ZuwryjZbFXFun
  ./redacteur add feed
URL: https://ahmet.im/blog/feed/rss.xml
Name: /home/alp
Group: dev
{"success":true,"feed":{"id":"34897cf8-8ea2-4601-93ca-74a59f7bb16e","name":"/home/alp","url":"https://ahmet.im/blog/feed/rss.xml","group":"dev"},"message":""}% 

@mrusme mrusme closed this as completed Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants