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

Release 1.0.0 #3

Merged
merged 60 commits into from
Mar 25, 2019
Merged

Release 1.0.0 #3

merged 60 commits into from
Mar 25, 2019

Conversation

alex-galych
Copy link
Contributor

@alex-galych alex-galych commented Mar 20, 2019

Please confirm

  • No new OS components - or they have been approved by the legal department

Documentation

  • Functional documentation provided or in progress?
  • Integration guide for projects provided or not needed?
  • Migration guides for all contained majors provided or not needed?

Release Table

Module Release Type Constraint Updates
Episerver major (new)

Release Notes

Added Episerver mail service functionality


Module Episerver

Change log

Initial Release

Added Episerver mail service functionality to send Customer, Order and Newsletter emails through a third-party service

profuel and others added 30 commits February 1, 2018 17:47
MISC initial composer.json
…optivo into feature/eco-1681/dev-optivo

# Conflicts:
#	src/SprykerEco/Zed/Optivo/Business/OptivoFacade.php
#	src/SprykerEco/Zed/Optivo/Business/OptivoFacadeInterface.php
#	src/SprykerEco/Zed/Optivo/Communication/Plugin/Oms/Command/OptivoNewOrderPlugin.php
@alex-galych alex-galych changed the title Dev Release 1.0.0 Mar 20, 2019
@alex-galych
Copy link
Contributor Author

Fix comments from #2 before release

@alex-galych alex-galych removed the request for review from SerhiiBabich March 20, 2019 15:14
*/
interface EpiserverConstants
{
public const CONFIGURATION_DEFAULT_MAILING_ID_LIST = 'EPISERVER:CONFIGURATION_DEFAULT_MAILING_ID_LIST';

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be marked with @api tag, because this type of constants are part of the public api

@alex-galych alex-galych merged commit 470e9be into master Mar 25, 2019
@alex-galych alex-galych deleted the dev branch June 5, 2019 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants