-
Notifications
You must be signed in to change notification settings - Fork 34
/collection/{slug}: exception happens when user adds a series to collection that already has this series #986
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
Comments
In fact we should revert the changes from |
Just removing an unique constraint shouldn't introduce bugs but a full support for having many series in a collection will do that:
|
^^^ these points will be addressed later |
I checked out a few articles to grasp the concept of rollback, but I don't understand it completely.
|
Should I follow this approach? |
Yes, something like this. Are you sure that we need to provide Also, could we use rollback section with a reference to the original changeset? Like in the example at https://www.liquibase.org/documentation/rollback.html
|
https://www.baeldung.com/liquibase-rollback, I have read this article for understanding the rollback concept. I am not sure how it fits our need. |
Most likely we won't need/use it but it's good to have it anyway, just in case. Also, it serves for documentation purpose in this case. |
@mukeshk I'm going to fix this if you don't mind. |
If you have not fixed it yet. Can I work on this tomorrow.
I have been stuck few things lately.
Thank you.
…On Sat 9 Nov, 2019, 3:49 AM Slava Semushin, ***@***.***> wrote:
@mukeshk <https://github.com/mukeshk> I'm going to fix it if you don't
mind.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#986?email_source=notifications&email_token=AAJPE6AKYX7QYY4QKD473CTQSXQXZA5CNFSM4GLFIHEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDTQTUA#issuecomment-552012240>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJPE6H7GJGS4BDQEJNBWFLQSXQXZANCNFSM4GLFIHEA>
.
|
@mukeshk Sure, thank you for the help! It's not urgent and it can be fixed during the next week. But it's definitely should be included in 0.4.2 release that is planned on Dec 15 and this issue is just the first step in order to have a support for multiple series in a collection. That means that we can't fix it just a day before the deadline. |
Sure. Slava :) is there any major in Dev environment ?
…On Sat 9 Nov, 2019, 10:27 PM Slava Semushin, ***@***.***> wrote:
@mukeshk <https://github.com/mukeshk> Sure, thank you for the help!
It's not urgent and it can be fixed during the next week. But it's
definitely should be included in 0.4.2 release that is planned on Dec 15
and this issue is just the first step in order to have a support for
multiple series in a collection. That means that we can't fix it just a day
before the deadline.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#986?email_source=notifications&email_token=AAJPE6FYR4X6QWJ4WK5BPQTQS3TZLA5CNFSM4GLFIHEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDUKL2Q#issuecomment-552117738>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJPE6E4YSPHQEYR3MFLCV3QS3TZLANCNFSM4GLFIHEA>
.
|
I'm not sure that I understood the question. What do you mean? Is there something more important than this task? |
No I was speaking any change in framework or libraries when I take an
update. Do I need java version or something like that?
…On Sat 9 Nov, 2019, 10:42 PM Slava Semushin, ***@***.***> wrote:
is there any major in Dev environment ?
I'm not sure that I understood the question. What do you mean? Is there
something more important than this task?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#986?email_source=notifications&email_token=AAJPE6GGBKYGPF546MXKUCDQS3VQVA5CNFSM4GLFIHEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDUKVII#issuecomment-552118945>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJPE6C2XJZZ6UZKBX45K3DQS3VQVANCNFSM4GLFIHEA>
.
|
Or all is same as before
…On Sun 10 Nov, 2019, 12:05 AM Mukesh Katariya, ***@***.***> wrote:
No I was speaking any change in framework or libraries when I take an
update. Do I need java version or something like that?
On Sat 9 Nov, 2019, 10:42 PM Slava Semushin, ***@***.***>
wrote:
> is there any major in Dev environment ?
>
> I'm not sure that I understood the question. What do you mean? Is there
> something more important than this task?
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#986?email_source=notifications&email_token=AAJPE6GGBKYGPF546MXKUCDQS3VQVA5CNFSM4GLFIHEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDUKVII#issuecomment-552118945>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AAJPE6C2XJZZ6UZKBX45K3DQS3VQVANCNFSM4GLFIHEA>
> .
>
|
I don't remember such changes so everything is the same. Ah, yes, we run our integration tests against PostgreSQL now (see #1140). Also Travis CI is failing frequently without showing the logs :-( |
@mukeshk weekly ping ;) don't hesitate to say "no" if you don't have time. It's open source and here are no obligations. |
Hi Slava,
Actually I made a new branch. I wanted to contribute but somehow I am
getting pulled into things.
I think you can take this up I will connect with you as I get some
bandwidth.
I apologise for the delay on my part.
…On Sat 16 Nov, 2019, 9:37 PM Slava Semushin, ***@***.***> wrote:
@mukeshk <https://github.com/mukeshk> weekly ping ;) don't hesitate to
say "no" if you don't have time. It's open source and here are no
obligations.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#986?email_source=notifications&email_token=AAJPE6H425PBLBPOCJTN7A3QUALCLA5CNFSM4GLFIHEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEHUVNQ#issuecomment-554650294>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJPE6CDFJYUBI4PRMUZQA3QUALCLANCNFSM4GLFIHEA>
.
|
Mukesh, thank you for the update and have a nice weekend! |
Solution:
The text was updated successfully, but these errors were encountered: