Skip to content

Conversation

@dastgirp
Copy link
Member

@dastgirp dastgirp commented Jul 26, 2018

Pull Request Prelude

Changes Proposed

Some of the strings were hardcoded in the source,
moved them to messages.conf

Affected Branches:

Issues addressed:

#1282

Known Issues and TODO List

Let me know if there are any more missing messages

@HerculesWSAPI
Copy link
Contributor

This change is Reviewable

@dastgirp dastgirp force-pushed the clif_messages branch 2 times, most recently from a0e6a09 to 7091870 Compare July 26, 2018 07:15
@dastgirp dastgirp added type:enhancement Issue describes an enhancement or feature that should be implemented component:other Any issues that don't fall in the explicitly described components labels Jul 29, 2018
@dastgirp dastgirp requested review from 4144 and Helianthella October 13, 2018 11:28
@dastgirp dastgirp requested a review from Asheraf October 13, 2018 11:35
@dastgirp
Copy link
Member Author

dastgirp commented Oct 2, 2019

Conflicts Fixed

@MishimaHaruna MishimaHaruna merged commit ad46000 into HerculesWS:master Oct 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:other Any issues that don't fall in the explicitly described components type:enhancement Issue describes an enhancement or feature that should be implemented

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants