-
-
Notifications
You must be signed in to change notification settings - Fork 499
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
Work done to allow user to update table cell. #546
Conversation
@RedEyedDog Could you rebase this against upstream |
I'm not sure what you mean sorry is it out of date with master? |
@RedEyedDog Yes exactly. Sync your |
Hey its telling me I am up to date ? |
17d050a
to
9cc7a9c
Compare
@patriksvensson ok I have done that. Sorry this will be my first contribution to an open source project. POP MY CHERRY ;) |
@RedEyedDog Great! Running CI again. Welcome to the world of OSS btw! 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (looks good to me) 👍
Thanks for all your help :) till next time |
Issue URL: #544
Hope everything is up to standard.
let me know of any changes.