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

Improve the management of related content #4499

Merged
merged 1 commit into from
Jun 25, 2021
Merged

Conversation

taitus
Copy link
Member

@taitus taitus commented Apr 30, 2021

References

Related PR: #3780
Closes #4498

Objectives

We detect when we add related content that already exists and display a slightly clearer message.

@taitus taitus self-assigned this Apr 30, 2021
@taitus taitus added the Bug label Apr 30, 2021
@taitus taitus added this to Reviewing in Consul Democracy via automation Apr 30, 2021
Consul Democracy automation moved this from Reviewing to Testing Jun 25, 2021
It looks like this bug was introduced in commit 7ca55c4. Before that
commit, a message saying "You added a new related content" was shown,
but (as expected) the related content wasn't added twice.

We now check this case and add a slightly clearer message.
@javierm javierm merged commit 92ad957 into master Jun 25, 2021
Consul Democracy automation moved this from Testing to Release 1.4.0 Jun 25, 2021
@javierm javierm deleted the related-content-exist branch June 25, 2021 15:31
@javierm javierm self-assigned this Jun 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error 422 when try to add existing related content
2 participants