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

Fix maxLength support #50

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

chriskl
Copy link

@chriskl chriskl commented Mar 18, 2021

No description provided.

@chriskl chriskl marked this pull request as ready for review March 18, 2021 07:38
@chriskl
Copy link
Author

chriskl commented Mar 18, 2021

How does this look? I've not tested it in anger yet

@AlexanderMann
Copy link
Collaborator

Hey @chriskl, sorry for just now seeing this, truly, apologies. This, on the whole looks good to me. I would just want @awm33 to sign off on this as I think he's actively leveraging this repo more than myself at this moment. Lemme see if I can poke him and get this moving for you.

@chriskl
Copy link
Author

chriskl commented Apr 18, 2021

Only thing I think is that you might want a major version bump because as soon as this is in master you'll get targets going crazy with column resizing if they are not clever about it.

@chriskl
Copy link
Author

chriskl commented Apr 19, 2021

Oops I thought this comment was on my PR for tap-salesforce.

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