Skip to content

DM-4574: Table (JS): text view#24

Merged
loitly merged 2 commits intodevfrom
DM-4574
Feb 29, 2016
Merged

DM-4574: Table (JS): text view#24
loitly merged 2 commits intodevfrom
DM-4574

Conversation

@loitly
Copy link
Copy Markdown
Contributor

@loitly loitly commented Feb 27, 2016

Added text view. Also, added the ability to save the table.

@robyww
Copy link
Copy Markdown
Contributor

robyww commented Feb 29, 2016

The code all looks fine. First pull request that I don't have anything to say about any line of code!

I am getting an exception:

TableStore:

    onUnmount() {
        const {tbl_id} = this.tableModel;   //<---- here, line 26
        this.handleAction(TblCntlr.TABLE_REMOVE, {tbl_id});
    }

The exception happens when I show the table in the tri-view and try to switch to another view, such as side-by-side.

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.

2 participants