-
Notifications
You must be signed in to change notification settings - Fork 130
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
Bump Version from 2.1.2 to 2.1.3, switch to woofmark plots2 branch #557
Conversation
Hi @NitinBhasneria do you think we could try resolving #553 and #475 to try to make reviewing these releases easier, before publishing this one? |
Yea, this would be great. |
Ok! I think our checklist for this is now:
However, I think we can push off the second list item to Great work, folks! |
@keshav234156 i think so; but let's make the top priority fixing the bugs introduced in |
#519 is now merged into main! So, let's add it to the release notes above. Thanks! |
Let's include #521 ! |
#564 is also merged now! |
Hi @publiclab/editor folks! How are we doing here? We need to prioritize this in order to publish our first release, and won't that feel great!? Can you check in on the blocking issues related to my comment above? Thank you so much! |
ok @jywarren having a look at this. |
Hi @NitinBhasneria - so after merging #587 I merged #588 which bumped the NPM version. Let's change this one to |
package.json
Outdated
@@ -1,6 +1,6 @@ | |||
{ | |||
"name": "publiclab-editor", | |||
"version": "2.1.1", | |||
"version": "2.1.2", |
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.
OK! And once this merges, can you create a new Release with the release notes here? I think we need to add a few more updates from https://github.com/publiclab/PublicLab.Editor/pulse/monthly as well to summarize other changes from folks. @publiclab/editor who else has had changes merged? Can we include #568 @keshav234156 ?
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.
yes, we can include the #568.
Also, I have updated the list have a look.
Very exciting 🎉 |
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.
@NitinBhasneria can you run npm install
by checking into this branch? There should be changes in package-lock.json
also I believe.
yea, sure. |
Looks good!! 🎉 🎉 |
package-lock.json
Outdated
@@ -13298,10 +13301,11 @@ | |||
"dev": true | |||
}, | |||
"woofmark": { | |||
"version": "git://github.com/jywarren/woofmark.git#12e2c1d4559268e2de605ad6084d5dce24ea42c6", | |||
"version": "git://github.com/jywarren/woofmark.git#39237b798308e6855acde1a938c18a8e4da62fce", |
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.
Only this should had changed, everything else should be same.
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.
Hi all, thanks! Looks like tests failed, could it be due to the woofmark changes? |
Huh. The error is: https://travis-ci.com/github/publiclab/PublicLab.Editor/jobs/370789599#L211 I'll restart it but is there any possibility of an issue in
|
And noting we had forgotten to build So this now becomes |
Hi all, i think these should be added here too: |
Resolved a conflict in package-lock.json - can folks confirm that this is correct by testing it in GitPod? Thanks! |
Ah, let's resolve this critical main image upload bug first, then this should be good for final approval and publication! #593 |
As well as this formatting issue to try to get the main image module a little more compact - that would be really awesome! #592 -- any volunteers? |
Thanks @shreyaa-sharmaa for submitting this PR for the button styling!!! #594 Awesome, almost there!! |
Hi all! With Compact Design for main Image Module #594 by @shreyaa-sharmaa, we are ready to publish this!!!! 🎉 🎉 🎉 🎉 |
I'm going to merge this and publish it! 🎉 🎉 🎉 🎉 🎉 |
Technically upon review this would be a minor version number bump of |
And running plots2 tests here! publiclab/plots2#8297 |
Fixes
woofmark
to pull in fixes from latest https://github.com/jywarren/woofmark/After publiclab/plots2#8285 merges, we can publish this!