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

Play 2.7 support (and cross build for Scala 2.13.0-M5) #115

Closed
mkurz opened this issue Feb 5, 2019 · 23 comments
Closed

Play 2.7 support (and cross build for Scala 2.13.0-M5) #115

mkurz opened this issue Feb 5, 2019 · 23 comments

Comments

@mkurz
Copy link
Contributor

mkurz commented Feb 5, 2019

Please add support for Play 2.7.
https://blog.playframework.com/play-2-7-0-is-here/
https://www.playframework.com/documentation/2.7.x/Highlights27
https://www.playframework.com/documentation/2.7.x/Migration27

Please also publish builds for Scala 2.13.0-M5. (Add it to crossScalaVersions, also please update to latest 2.12 and latest 2.11)
Your probably have to set PlayKeys.playOmnidoc := false in build.sbt to make it work.

Also it could be that you have to adjust some twirl templates because now curly brackets are needed in some cases.
E.g. twirl code like

@something.map { case SomeThing(foo) =>
	some html
}

becomes this:

@something.map { case SomeThing(foo) => {
	some html
}}
@mkurz mkurz mentioned this issue Feb 5, 2019
@adrianhurt
Copy link
Owner

Hi @mkurz! I've just finally released the snapshots for 1.5 versions with Play 2.7 (the code is on branch develop). Please, try to test it. I've just found a lot of warnings like this one:

[warn] /path/to/play-bootstrap/play27-bootstrap4/module/app/views/b4/buttonType.scala.html:4:16:
Passing an explicit array value to a Scala varargs method is deprecated (since 2.13.0) and will result in a defensive copy;
Use the more efficient non-copying ArraySeq.unsafeWrapArray or an explicit toIndexedSeq call
[warn] }(fc, msgsProv)

I'll try to check them as soon as possible and to update the docs as well.

The snapshots versions are:

  • 1.5-P27-B4-SNAPSHOT
  • 1.5-P26-B4-SNAPSHOT

@mkurz
Copy link
Contributor Author

mkurz commented Feb 20, 2019

@adrianhurt Thanks! I will check it out as soon as possible! Again thank you very much for your efforts!

@bjoernhaeuser
Copy link

I tested the 2.7 snapshot and it looks very good to me, though I only have a minor use case - two simple forms.

@mkurz
Copy link
Contributor Author

mkurz commented May 30, 2019

I think we should wait with the next release until next week because then Scala 2.13 will be released and also Play will be published for Scala 2.13 - meaning also play-bootstrap can then also published for Scala 2.13 as well.

@adrianhurt
Copy link
Owner

Yes, I think it’s better to wait as well. I’d only update something to fix bugs if needed, but please, test it if you all have time.

@bjoernhaeuser
Copy link

Scala 2.13 is now released, looking forward to the release of play-boostrap. Thank you!

@mkurz
Copy link
Contributor Author

mkurz commented Jun 19, 2019

@bjoernhaeuser yes but we need Play to support Scala 2.13 first

@ahjohannessen
Copy link

ahjohannessen commented Jun 21, 2019

@adrianhurt
Copy link
Owner

Great! I’ll try to update it this weekend!

@mkurz
Copy link
Contributor Author

mkurz commented Jun 21, 2019

@adrianhurt I will submit a pull request later today with an important fix. Please wait for it, thanks!

@mkurz
Copy link
Contributor Author

mkurz commented Jun 22, 2019

Please don't release yet, as I said will provide a pull request with a bugfix very soon!

@adrianhurt
Copy link
Owner

Don't worry, I'll wait for it.

@mkurz
Copy link
Contributor Author

mkurz commented Jun 22, 2019

@adrianhurt Done, see #121 (also maybe merge #120 before).
Let me know if you need help. Thanks!

@adrianhurt
Copy link
Owner

Hi! I've just released versions 1.5-P2X-BX-RC-SNAPSHOT with Play 2.7.3 (note the -RC). If you don't see any problem I'll release the final one during the next week.

I've noted the syntax 'foo for a symbol has been deprecated, so I suppose in the version 1.6 I'll substitute (maybe) every symbol by a simple string.

@mkurz
Copy link
Contributor Author

mkurz commented Jun 24, 2019

@adrianhurt Just did a small test, for me it looks good, thanks!

@bjoernhaeuser
Copy link

Same here :) small test was successful! Thanks!

@adrianhurt
Copy link
Owner

Hey guys! It's finally released

@mkurz
Copy link
Contributor Author

mkurz commented Jul 3, 2019

Thanks!

@mkurz
Copy link
Contributor Author

mkurz commented Sep 9, 2019

I've noted the syntax 'foo for a symbol has been deprecated, so I suppose in the version 1.6 I'll substitute (maybe) every symbol by a simple string.

That is what Play itself will do as well: See playframework/playframework#9447 and playframework/playframework#9488. Therefore 'key -> "value" will become "key" -> "value"...

@fooblahblah
Copy link

fooblahblah commented Sep 25, 2019

Any updates on the Symbol deprecation thing? I'm trying to get our app compiling under 2.13.1 and the last issue is the Play template using literal Symbols. I migrated to the Symbol("foo") syntax, but get some implicit errors, but maybe it's unrelated? BTW, I'm using 1.5.1-P27-B3

could not find implicit value for parameter fc: views.html.b3.package.B3FieldConstructor
[error]               @b3.select(catForm("categoryType"), categoryTypeSelectOptions, Symbol("_label") -> "Category Type")

@mkurz
Copy link
Contributor Author

mkurz commented Sep 25, 2019

@fooblahblah No updates yet. I think the implicit error you face is not related to symbols.

@fooblahblah
Copy link

Thanks @adrianhurt. I'm starting to think the same. I need to figure out why these usages started breaking when I changed versions of the library.

@mkurz
Copy link
Contributor Author

mkurz commented Nov 20, 2019

See #124 for the symbol issue.

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

No branches or pull requests

5 participants