search

IITC-CE / ingress-intel-total-conversion

intel.ingress.com total conversion user script with some new features. Should allow easier extension of the intel map.
https://iitc.app
ISC License
290 stars 109 forks source link

v0.29 #98

Closed johnd0e closed 5 years ago

johnd0e commented 5 years ago

@modos189 So what features do you want to include in nearest release? Would you like to use Project#1 to track the progress?

modos189 commented 5 years ago

I think now there are all the changes I planned. I need to start making a list for merge

johnd0e commented 5 years ago

That list is in the Project 1 already, try it.

johnd0e commented 5 years ago

P.S. And I'd like to finish #110 before release

modos189 commented 5 years ago

Ok. Indeed, it is better to make such changes before the auto-update works.

johnd0e commented 5 years ago

@modos189 Some of issues require your feedback: https://github.com/IITC-CE/ingress-intel-total-conversion/issues?utf8=%E2%9C%93&q=label%3Aattention

johnd0e commented 5 years ago

I've finished planned changes for this release. Next: we should resolve a couple of issues that needs your attention.

Edit: we need to solve scripts versioning issue.

modos189 commented 5 years ago

I suggest to test again and release (for example) tomorrow

johnd0e commented 5 years ago

Sure, but I still want to resolve versioning issues before release: https://github.com/IITC-CE/ingress-intel-total-conversion/issues/99#issuecomment-468583737

johnd0e commented 5 years ago

https://github.com/IITC-CE/iitc.modos189.ru/issues/1 is also important for release

modos189 commented 5 years ago

IITC-CE/iitc.modos189.ru#1 is also important for release

This is a useful change, but not a blocking release. This can be implemented soon.

johnd0e commented 5 years ago

Well, download page looks not ok (after renames), and it's bad for new release. We should at least make temporary category correction now. I could do it tomorrow.

johnd0e commented 5 years ago

There 2 PRs left unmerged. Why?

modos189 commented 5 years ago

I thought they weren't ready. Then, in the next release.

johnd0e commented 5 years ago

Renames should be applied as soon as possible

johnd0e commented 5 years ago

I suppose we should release 0.29.1

modos189 commented 5 years ago

If do this only for desktop version, nothing will break?

johnd0e commented 5 years ago

I suppose all will be ok, but why not release mobile too?

P.S. Actually I need updated mobile release, because I am unable to use test-builds apk due of #12.

johnd0e commented 5 years ago

In release notes there should be some warning:

Desktop users: additional one-time action required when updating to current release!

Some plugins are renamed, and do not replace old versions automatically.
So it's necessary to delete old versions manually.

Sorry for the inconvenience.
modos189 commented 5 years ago

Added to description of github release

johnd0e commented 5 years ago

What about to add this to news stream on site (and tg channel)?

modos189 commented 5 years ago

I don’t want to write there, because then news about release 0.29 will not be displayed on the site :( This is not so important; not so many people were updated in a day.

johnd0e commented 5 years ago

I don’t want to write there, because then news about release 0.29 will not be displayed on the site :(

Isn't possible to add this warning to existent description of 0.29?

johnd0e commented 5 years ago

Build Status