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

#432 #434

Merged
merged 14 commits into from
Jun 25, 2021
Merged

#432 #434

merged 14 commits into from
Jun 25, 2021

Conversation

jijojosephk
Copy link
Collaborator

I've implemented the fix. Try to merge the PR.

@simonvanderveldt
Copy link

Fixes #432

Copy link
Owner

@IsmaelMartinez IsmaelMartinez left a comment

Choose a reason for hiding this comment

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

Left a few comments mainly on function sizes. Nothing serious but I found writing smaller functions makes it easier to maintain. Main complainer is code beat https://codebeat.co/projects/github-com-ismaelmartinez-teams-for-linux-develop/pull_requests/476509 that keeps emailing me every build ! Thanks a lot for that great job

app/browser/desktopShare/chromeApi.js Outdated Show resolved Hide resolved
app/browser/rightClickMenuWithSpellcheck.js Show resolved Hide resolved
app/mainAppWindow/index.js Outdated Show resolved Hide resolved
app/mainAppWindow/index.js Outdated Show resolved Hide resolved
app/streamSelector/browser.js Outdated Show resolved Hide resolved
app/streamSelector/browser.js Outdated Show resolved Hide resolved
app/streamSelector/index.css Show resolved Hide resolved
app/streamSelector/index.html Show resolved Hide resolved
app/streamSelector/index.js Outdated Show resolved Hide resolved
app/streamSelector/index.js Outdated Show resolved Hide resolved
Copy link
Owner

@IsmaelMartinez IsmaelMartinez left a comment

Choose a reason for hiding this comment

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

Last, but not least, can you increase the https://github.com/IsmaelMartinez/teams-for-linux/blob/develop/package.json#L3 version number? just increase it to 1.0.8 (mainly to avoid getting close to have the same 1.3.x version as MS).

Once that build is ready, I would release it as a pre-release version so people can check the build ones.

Thanks a lot for your hard work!

@jijojosephk
Copy link
Collaborator Author

Last, but not least, can you increase the https://github.com/IsmaelMartinez/teams-for-linux/blob/develop/package.json#L3 version number? just increase it to 1.0.8 (mainly to avoid getting close to have the same 1.3.x version as MS).

Once that build is ready, I would release it as a pre-release version so people can check the build ones.

Thanks a lot for your hard work!

@IsmaelMartinez , I've updated the version.

@IsmaelMartinez IsmaelMartinez merged commit 741a5ca into IsmaelMartinez:develop Jun 25, 2021
@jijojosephk jijojosephk deleted the issue-432 branch June 26, 2021 06:35
@IsmaelMartinez
Copy link
Owner

IsmaelMartinez commented Jun 26, 2021 via email

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

Successfully merging this pull request may close these issues.

None yet

4 participants