Deprecate net.liftweb.builtin.snippet.A #1241

Closed
wants to merge 1 commit into
from

Projects

None yet

4 participants

@fmpwizard
Lift Web Framework member

As discussed here https://groups.google.com/d/topic/liftweb/cM8gXrHK3f0/discussion

  • Deprecate builtin.snippet.A ** (As Jeppe suggested: "remove the definition from LiftRules.snippetDispatch and deprecate the class)
  • Replace any internal use of it for S.fmapFunc
@fmpwizard fmpwizard fixed #1241 - Deprecate net.liftweb.builtin.snippet.A
** Changed SHtml.s not to use the builtin A object, but to use fmapFunc instead
40a3528
@fmpwizard fmpwizard added a commit that closed this pull request Mar 24, 2012
@fmpwizard fmpwizard fixed #1241 - Deprecate net.liftweb.builtin.snippet.A
** Changed SHtml.s not to use the builtin A object, but to use fmapFunc instead
40a3528
@fmpwizard fmpwizard closed this in 40a3528 Mar 24, 2012
@fmpwizard
Lift Web Framework member

For some reason when I push a branch to github, the ticket gets close, so reopening it.

@fmpwizard fmpwizard reopened this Mar 24, 2012
@fmpwizard
Lift Web Framework member

The commit message
"** Changed SHtml.s not to use" should have said
""** Changed SHtml.a not to use"

(I changed SHtml.a not SHtml.s )

@lkuczera
Lift Web Framework member

+1

@fmpwizard fmpwizard added a commit that closed this pull request Mar 29, 2012
@fmpwizard fmpwizard fixed #1241 - Deprecate net.liftweb.builtin.snippet.A
** Changed SHtml.a not to use the builtin A object, but to use fmapFunc instead
b8983ff
@fmpwizard fmpwizard closed this in b8983ff Mar 29, 2012
@indrajitr
Lift Web Framework member

Merged to master, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment