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

🐛 BUG: link marked as existing if submitted to private squad first #1109

Open
1 task done
palmiak opened this issue Apr 5, 2024 · 3 comments
Open
1 task done

Comments

@palmiak
Copy link

palmiak commented Apr 5, 2024

What went wrong? 🤔

If I submit a link to private squad and later I try to submit it to community picks I get info that this link already exists.

Expected Behavior

I should be added normally - not treated as a duplicate.

Steps to Reproduce Issue

1. Add a link that wasn't added before to a private squad
2. Click "submit new link" and try adding the same link - you should get info that it already exists.

Solution Proposed

Links that were added in squads, shouldn't be saw as duplicates when submitting them into community picks.

Screenshots

NA

Environment

NA

Browsers

Microsoft Edge

OS

Windows

Version of daily.dev

NA

Additional Context

NA

Code of Conduct

  • I follow the conditions of this project.
@idoshamun
Copy link
Member

We're aware of this issue but due to technical challenges it's not prioritized for fix but may be fixed in the future. We're doing some architectural changes to help us address such edge cases better. I hope you understand 🙏

@palmiak
Copy link
Author

palmiak commented Apr 7, 2024 via email

@idoshamun
Copy link
Member

So I must remember to first submit globally and later locally.

That should fix it indeed!

I can imagine that every link should have a flag is_global.

A bit more complicated than this. Links that are added via squad are assigned to an unknown source and are not visible on their own, only via their squad post variation. Community picks are assigned with community source and it currently doesn't support adding an existing link. To support this use case we will need to change the source of the post which is currently immutable and changing it has a lot of side effects. Plenty of technical decisions that led us to this situation 🥵

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants