Skip to content
This repository has been archived by the owner on Jul 7, 2024. It is now read-only.
/ Orion Public archive

Shows the activity window when adding a new file #142

Merged
merged 8 commits into from
Jul 2, 2018

Conversation

kernelwhisperer
Copy link
Contributor

What changed?

Added the activities window.
This is a WIP, but please take a look before I continue in this direction.

@kernelwhisperer kernelwhisperer changed the title [WIP] Bugfix/157 progress bar while adding Bugfix/157 progress bar while adding Jun 29, 2018
@kernelwhisperer
Copy link
Contributor Author

ready for review :)

@koalalorenzo koalalorenzo changed the title Bugfix/157 progress bar while adding Shows the activity window when adding a new file Jun 30, 2018
Copy link
Member

@koalalorenzo koalalorenzo left a comment

Choose a reason for hiding this comment

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

We have also some tests that are failing... 😮

app/api.js Outdated
const filename = parse(path).base
const uuid = uuidv4()

if (size >= ACTIVITIES_WINDOW_THRESHOLD) {
Copy link
Member

Choose a reason for hiding this comment

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

can we add few lines of comments to describe why/what?

app/api.js Outdated
app.emit('show-activities-window')
}

const options = {
Copy link
Member

Choose a reason for hiding this comment

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

also here: comments

app/index.js Outdated
@@ -286,6 +292,44 @@ app.on('start-orion', () => {
startWelcome().then(startOrion)
})

app.on('show-activities-window', () => {
Copy link
Member

Choose a reason for hiding this comment

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

Few comments here?

Copy link
Member

@koalalorenzo koalalorenzo left a comment

Choose a reason for hiding this comment

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

Well done 👏

@kernelwhisperer kernelwhisperer merged commit 72f3a40 into master Jul 2, 2018
@kernelwhisperer kernelwhisperer deleted the bugfix/157-progress-bar-while-adding branch July 2, 2018 19:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants