https://linen.dev logo
Join Slack
Powered by
Hiya, <@U01ADSJ1D5H>, is the multiple destinations...
# dev-frontend
c
Hiya, @Artem Astapenko, is the multiple destinations stuff ready for us to review and try out or are you still working on it?
🙌 1
a
Hi, everything is ready - I am spending some time on testing. We need to spend some time on testing and after that can merge
c
Okay!
Thanks.
a
btw, guys, you can test. Let me update review 🙂
You can leave comments directly in review if you find any errors and tomorrow I am going to check everything 1 more time and merge 🙂
c
okay. we should just test by pulling down this branch right: source-and-destination-pages-0.5.0?
a
Yep
c
I tested this a little bit. Looking good! I'm having trouble getting syncs to succeed, which I think is just because master needs to be pulled in (i don't think it's a frontend issue).
a
Ok. I will merge last mater and test one more time. Hope today we can merge this feature :)
c
Great! Agreed.
Let me know when you have merge master and I will test one more time quickly.
a
Done
c
tested it and it is looking good to me!
i think you should merge it when you are happy with it.
i noticed one bug, but let's fix it after you merge.
message has been deleted
the destination name wasn't showing after i added an existing source to an existing destination.
refreshing the page fixed it.
after the page was refresh it was good:
message has been deleted
a
Merged! 🎉 Btw, will you create task for the issue above?
c
yes!
i will.
2 Views
Open in Slack
PreviousNext
Powered by