Skip to content
This repository was archived by the owner on Apr 1, 2020. It is now read-only.

Conversation

@tavvy
Copy link
Contributor

@tavvy tavvy commented Jan 4, 2018

See https://github.com/Financial-Times/n-messaging-client readme for more info

Why install globally in n-ui?
If the end goal is to have consistent messaging across FT.com then it makes sense that it is an "easy to enable" n-ui feature.


This demonstrates how the new n-messaging-client could be integrated with n-ui globally.

It would be off by default. To enable an application must invoke n-express with the withMessaging option (which will load the inject the templates and load the presenter). And enable the messaging feature in nUiConfig to load in the client side js.

const app = module.exports = express({
	withMessaging: true,
});

app.locals.nUiConfig = {
	preset: 'complete',
	features: {
		messaging: true
	}
};

@tavvy tavvy force-pushed the at/n-messaging-client branch from 3636115 to 52fb5aa Compare January 4, 2018 11:36
tavvy added 2 commits January 4, 2018 14:58
 🐿 v2.6.0
 🐿 v2.6.0
@tavvy tavvy closed this Mar 2, 2018
@doramatadora doramatadora deleted the at/n-messaging-client branch June 26, 2018 20:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants