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

Add support to run custom server logic #22

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Add support to run custom server logic #22

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 23, 2015

This adds support to run custom server logic with the help of revel/revel#879.

@brendensoares
Copy link
Member

More interesting code. Will look deeper soon!

@brendensoares
Copy link
Member

I think we should flesh this out in more detail before we accept this PR. There are likely many ways to accomplish this task.

@pedromorgan
Copy link
Member

Na stuff the RFC.. any good idea..
We just stick it in developer and clearly flag as "experimental"

@brendensoares
Copy link
Member

@pedromorgan how will we clearly flag it? How will we document how it works for the Revel docs? How will we know when it doesn't break existing features?

We can't just throw in code willy nilly. We have to make it hard to change Revel and easy to improve it. There must be a process for evaluating incoming code changes to decide if it's the right solution for Revel's goals.

@brendensoares brendensoares added this to the v0.13 milestone Apr 20, 2015
@jeevatkm jeevatkm modified the milestones: Backlog, v0.13 May 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants