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

New Feature: Package pane functions become a command (pane) and implement snapshot saved. #183

Merged
merged 2 commits into from Apr 8, 2015

Conversation

samejack
Copy link
Contributor

@samejack samejack commented Apr 8, 2015

No description provided.

@audreyt
Copy link
Owner

audreyt commented Apr 8, 2015

Wow, awesome! 🎆 LGTM, I'll review it later tonight for merging.

audreyt added a commit that referenced this pull request Apr 8, 2015
New Feature: Package pane functions become a command (pane) and implement snapshot saved.
@audreyt audreyt merged commit e4fa495 into audreyt:master Apr 8, 2015
audreyt added a commit that referenced this pull request Apr 21, 2015
Revert "New Feature: Package pane functions become a command (pane) and implement snapshot saved."

This reverts commit 9c676c0.

Conflicts:
	socialcalc-3.js
@audreyt
Copy link
Owner

audreyt commented Apr 21, 2015

Hi @samejack , sorry for the revert — it seems with this change, loading an existing spreadsheet, insert a new row and pasting some text will clear the spreadsheet. Would you like to see if you can reproduce this error?

@samejack
Copy link
Contributor Author

I'll check tomorrow, thanks.

samejack added a commit to samejack/ethercalc that referenced this pull request Apr 23, 2015
@samejack
Copy link
Contributor Author

Hello @audreyt, i have fixed this issue. How do i create Pull Request !? Thanks.

audreyt added a commit that referenced this pull request Apr 23, 2015
This reverts commit 9771f34.

Conflicts:
	manifest.appcache
audreyt added a commit that referenced this pull request Apr 23, 2015
@audreyt
Copy link
Owner

audreyt commented Apr 23, 2015

That's OK, I have manually applied the fix. I'll test it more locally and make a release in a few days. Thanks for the prompt fix!

@samejack
Copy link
Contributor Author

Thank you.

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

2 participants