Skip to content

Conversation

@taooceros
Copy link
Member

fix #354

@taooceros taooceros marked this pull request as draft February 22, 2021 03:48
@taooceros taooceros marked this pull request as ready for review February 22, 2021 04:12
@taooceros taooceros added the bug Something isn't working label Feb 22, 2021
@taooceros taooceros added this to the 1.8.0 milestone Feb 22, 2021
@taooceros taooceros self-assigned this Feb 22, 2021
@taooceros taooceros modified the milestones: 1.8.0, 1.7.3 Feb 22, 2021
@taooceros taooceros mentioned this pull request Feb 22, 2021
@taooceros
Copy link
Member Author

Todo: move http proxy initialization earlier before plugin initialization.

@jjw24 jjw24 modified the milestones: 1.7.3, 1.7.2 Feb 22, 2021
@jjw24
Copy link
Member

jjw24 commented Feb 22, 2021

is this ready to go? @taooceros

@taooceros
Copy link
Member Author

Todo: move http proxy initialization earlier before plugin initialization.

Let me finish this first. I just get my computer.

@taooceros
Copy link
Member Author

Done, please review @jjw24

Comment on lines 80 to 82



Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol you love your spaces dont you 😛

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah 🤣🤣🤣 I love to separate different parts of code doing different with spaces🤣🤣🤣

class HttpTest
{
[Test]
public void TestSettingUpdate()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you give this a bit more description please, Given-When-Then

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, I will add comment.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey if it's ok, i will update it for you, sorry i need to be afk soon and i want to get this release ready, so when you are free have a look at 1.7.2 and approve if all good, i will release it later

Copy link
Member Author

@taooceros taooceros Feb 22, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah sure, if you would like to do it, it will be great. I just have dinner so haven't finish that part.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will approve 1.7.2 once this has been merged. 🥰

@jjw24 jjw24 enabled auto-merge February 22, 2021 10:28
@jjw24 jjw24 merged commit 234e3da into Flow-Launcher:dev Feb 22, 2021
@taooceros taooceros deleted the fixHttpError branch February 25, 2021 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

didn't work

2 participants