Hey Ebenezer,

hope this finds you well. (;

looking at the two stack traces you've provided here is my take on it.

The error while storing configuration is caused by a field in the database
'sender_number' being declared as not null, but no data is provided. In the
create method of SMSConfiguration you are not handling the sender number at
all. I would recommend to either use a builder pattern or just use plain
getter/setter to create the instance.

With regards to receiving multiple (3) events, this is caused by the way
acitvemq is configured in your service. The default
for activemq.concurrency is 3-10, if you set this to 1-1 it will only
listen with 1 concurrent thread.

Hope this helps.

Cheers

Markus



On Wed, Aug 22, 2018 at 1:40 PM Ebenezer Graham <[email protected]>
wrote:

> Thanks myrle,
>
> I have already created a demo email account called
> [email protected] and I created it for the sole purpose
> of testing emails for this service and for other contributors to use.
> Please advise me on this approach.
>
> Also, the password for [email protected] and akyencorp@gmail are
> no longer valid. I created app passwords and so I have already deleted
> those app passwords for my live account and are no longer valid.
>
> *Error:*
> In an attempt to store configurations from the repository I get this error:
> https://pastebin.com/WqvRwXvB
>
> Please check: shouldCreateSMSConfigurationEntity()
> https://github.com/ebenezergraham/fineract-cn-notifications/
> blob/develop/component-test/src/main/java/org/apache/
> fineract/cn/notification/TestSMSService.java
>
> I am able to retrieve stored configurations but I am unable to create a
> configuration and store. Same issue for email repository
>
> Second Issue: https://pastebin.com/LKz7HcGy
> I have managed to authenticate with identity and getting customer object,
> however, when I trigger an invent from the fims web app, I get 3
> invocations from the ActiveMQ. How can I resolve this issue? Are my
> assumptions correct to begin with?
>
>
> *At your service,*
>
> *Ebenezer Graham*
>
> *BSc (Hons) Computing*
>
>
> [image: EmailSignature.png]
>
> African Leadership University,
>
> Power Mill Road, Pamplemousses,
>
> Mauritius.
>
>
> ​
> *skype*:
> ​ebenezer.graham
> GitHub <https://github.com/ebenezergraham> | LinkedIn
> <https://www.linkedin.com/in/ebenezer-graham/> | Twitter
> <https://twitter.com/pactmart> | Facebook
> <https://www.facebook.com/pactmart>
> www.pactmart.com | Freelancing made easy.
>
>
> *“Talk is cheap, show me the code.” *- *Linus Torvalds*
>
>
>
> On 22 August 2018 at 15:10, Ebenezer Graham <[email protected]>
> wrote:
>
> > Thanks myrle,
> >
> > I have already created a demo email account called
> > [email protected] and I created it for the sole
> > purpose of testing emails for this service and for other contributors
> > to use. Please advise me on this approach.
> >
> > Also, the password for [email protected] and akyencorp@gmail
> are
> > not the real passwords. I created app passwords and so I have already
> > deleted those app passwords for my live account and are no longer valid.
> >
> > *Error:*
> > In an attempt to store configurations from the repository I get this
> error:
> >
> > Please check: shouldCreateSMSConfigurationEntity()
> >
> https://github.com/ebenezergraham/fineract-cn-notifications/blob/develop/
> > component-test/src/main/java/org/apache/fineract/cn/
> > notification/TestSMSService.java
> >
> > I am able to retrieve stored configurations but I am unable to create a
> > configuration and store. Same issue for email repository
> >
> > Second Issue: https://pastebin.com/LKz7HcGy
> > I have managed to authenticate with identity and getting customer object,
> > however, when I trigger an invent from the fims web app, I get 3
> > invocations from the ActiveMQ. How can I resolve this issue? Are my
> > assumptions correct to begin with?
> >
> >
> > *At your service,*
> >
> > *Ebenezer Graham*
> >
> > *BSc (Hons) Computing*
> >
> >
> > [image: EmailSignature.png]
> >
> > African Leadership University,
> >
> > Power Mill Road, Pamplemousses,
> >
> > Mauritius.
> >
> >
> > ​
> > *skype*:
> > ​ebenezer.graham
> > GitHub <https://github.com/ebenezergraham> | LinkedIn
> > <https://www.linkedin.com/in/ebenezer-graham/> | Twitter
> > <https://twitter.com/pactmart> | Facebook
> > <https://www.facebook.com/pactmart>
> > www.pactmart.com | Freelancing made easy.
> >
> >
> > *“Talk is cheap, show me the code.” *- *Linus Torvalds*
> >
> >
> >
> > On 22 August 2018 at 14:45, Myrle Krantz <[email protected]> wrote:
> >
> >> Hey Ebenezer,
> >>
> >> I was just reviewing your PR and realized that you had included
> >> passwords in it.  They look like they might be passwords to live
> >> accounts.  I strongly suggest you remove them from your latest PR.
> >>
> >> I also checked and it appears you have passwords in code that has
> >> already been merged.  If this is true, I suggest you change the
> >> password on the e-mail account: [email protected]
> >>
> >> In general, we don't change git history.  It's okay to make mistakes,
> >> and it's important to preserve the commit history of a project.  There
> >> are few exceptions I would make, but one of those is for passwords or
> >> other secrets.  But only if those passwords or other secrets can't
> >> easily be replaced.  I don't believe this is the case here.
> >>
> >> Beyond that, I'm working on putting together feedback on your code and
> >> will post that by the end of the day.
> >>
> >> Best Regards,
> >> Myrle
> >>
> >>
> >>
> >>
> >> On Wed, Aug 22, 2018 at 9:12 AM Ebenezer Graham
> >> <[email protected]> wrote:
> >> >
> >> > Hi Awasum,
> >> >
> >> > I have been able to resolve the authentication problem and now able to
> >> > retrieve customers.
> >> >
> >> > I have also written a component test for the email and sms service.
> >> >
> >> > In an attempt to clean up my repository and squash the commits I
> messed
> >> up
> >> > my repo hence I have deleted it and forked a new one.
> >> >
> >> > In light of this, would like to find out if it is possible to force
> >> merge
> >> > them so that apache repository takes commits from mine.  This is
> because
> >> > the one on Apache has some unwanted files such as the sample classes
> and
> >> > travis.yml. Please let me know if this is possible.
> >> >
> >> > Also, I have made new PRs. Please review it and provide feedback
> >> >
> >> > Looking forward to your response.
> >> >
> >> > *At your service,*
> >> >
> >> > *Ebenezer Graham*
> >> >
> >> > *BSc (Hons) Computing*
> >> >
> >> >
> >> > [image: EmailSignature.png]
> >> >
> >> > African Leadership University,
> >> >
> >> > Power Mill Road, Pamplemousses,
> >> >
> >> > Mauritius.
> >> >
> >> >
> >> >
> >> > *skype*:
> >> > ebenezer.graham
> >> > GitHub <https://github.com/ebenezergraham> | LinkedIn
> >> > <https://www.linkedin.com/in/ebenezer-graham/> | Twitter
> >> > <https://twitter.com/pactmart> | Facebook
> >> > <https://www.facebook.com/pactmart>
> >> > www.pactmart.com | Freelancing made easy.
> >> >
> >> >
> >> > *“Talk is cheap, show me the code.” *- *Linus Torvalds*
> >> >
> >> >
> >> >
> >> > On 9 July 2018 at 12:25, Awasum Yannick <[email protected]> wrote:
> >> >
> >> > > Hello Graham,
> >> > >
> >> > > You need to create a user, maybe use *operator* and make sure the
> >> user has
> >> > > permission to access customer (READ). Then you have to create a
> class
> >> which
> >> > > uses this user to connect and authenticate each time with the
> identity
> >> > > service before each customer information is retrieved.
> >> > >
> >> > > This user could be defined in the yml file of your service and then
> >> > > injected into the Authentication Class you create.
> >> > >
> >> > > Each event which is emitted most often has the tenant within the
> >> message.
> >> > > just get this tenant and check and set the *TenantContextHolder *if
> >> not
> >> > > already set just before you authenticate. Immediately after
> >> authentication,
> >> > > Set the *UserContextHolder* too
> >> > >
> >> > > Another point is to remember to trim or remove the quotes from the
> >> customer
> >> > > identifier before trying to get more contact details about  that
> >> customer:
> >> > > {String}.replaceAll("^\"|\"$", "")
> >> > >
> >> > > I hope this helps
> >> > >
> >> > > Thanks.
> >> > > Awasum Yannick
> >> > >
> >> > > On Tue, Jul 3, 2018 at 11:18 PM Ebenezer Graham <
> >> [email protected]>
> >> > > wrote:
> >> > >
> >> > > > Hello there,
> >> > > >
> >> > > > I would like to retrieve a customer after it has been created, or
> >> > > activated
> >> > > > etc.
> >> > > >
> >> > > > I have coded the logic to send the SMS and email after getting the
> >> > > contact
> >> > > > details, however, I have to get the customer to get the contact
> >> details.
> >> > > >
> >> > > > This is the error I get during my call:
> >> https://pastebin.com/zrebwE8v
> >> > > >
> >> > > > I would like to get clarifications on how to set the tenant and
> >> correctly
> >> > > > make authentication during a rest call.
> >> > > >
> >> > > > I would also like to get a better understanding of how to use the
> >> Anubis,
> >> > > > test and identity microservice in this case
> >> > > >
> >> > > > *At your service,*
> >> > > >
> >> > > > *Ebenezer Graham*
> >> > > >
> >> > > > *BSc (Hons) Computing*
> >> > > >
> >> > > >
> >> > > > [image: EmailSignature.png]
> >> > > >
> >> > > > African Leadership University,
> >> > > >
> >> > > > Power Mill Road, Pamplemousses,
> >> > > >
> >> > > > Mauritius.
> >> > > >
> >> > > >
> >> > > > *skype*:
> >> > > > ebenezer.graham
> >> > > > GitHub <https://github.com/ebenezergraham> | LinkedIn
> >> > > > <https://www.linkedin.com/in/ebenezer-graham/> | Twitter
> >> > > > <https://twitter.com/pactmart> | Facebook
> >> > > > <https://www.facebook.com/pactmart>
> >> > > > www.pactmart.com | Freelancing made easy.
> >> > > >
> >> > > >
> >> > > > *“Talk is cheap, show me the code.” *- *Linus Torvalds*
> >> > > >
> >> > > >
> >> > > >
> >> > > > On 19 May 2018 at 00:41, Ebenezer Graham <
> [email protected]>
> >> > > wrote:
> >> > > >
> >> > > > > Hi everyone,
> >> > > > >
> >> > > > > I am Ebenezer a student from the African Leadership University
> >> and I
> >> > > will
> >> > > > > be working on this project under the mentorship of Isaac Kamga.
> >> > > > >
> >> > > > > The idea is to use this thread to discuss the decisions being
> >> made on
> >> > > > this
> >> > > > > project so that the community has visibility on what's happening
> >> and
> >> > > most
> >> > > > > importantly give constructive feedback.
> >> > > > >
> >> > > > > *EventListeners:*
> >> > > > > This is the Doc
> >> > > > > <
> >> > > > https://docs.google.com/document/d/1kme5qrxOrO0WGUEvNgzv5XVkgWtpg
> >> > > 0lbX-CrBH4js7o/edit?usp=sharing
> >> > > > >
> >> > > > > containing the shortlisted events which I believe qualify for an
> >> Email
> >> > > or
> >> > > > > SMS notification. Please review and comment on the doc
> suggesting
> >> any
> >> > > > > addition or deletion or related feedback.
> >> > > > >
> >> > > > > *UI for the service:*
> >> > > > > This is a link <https://wireframe.cc/pro/pp/5fb6b2e61167290> to
> >> the
> >> > > > > wireframes for the UI of the service. Please give pointers on
> what
> >> > > could
> >> > > > be
> >> > > > > added and what should be taken out.
> >> > > > >
> >> > > > >
> >> > > > > *At your service*
> >> > > > > *,*
> >> > > > >
> >> > > > > *Ebenezer Graham*
> >> > > > >
> >> > > > > *BSc (Hons) Computing*
> >> > > > >
> >> > > > >
> >> > > > > [image: EmailSignature.png]
> >> > > > >
> >> > > > > Beau Plan Business Park | West Building
> >> > > > >
> >> > > > > Pamplemousses |  Mauritius
> >> > > > >
> >> > > > >
> >> > > > > <http://www.pactmart.com/>
> >> > > > > <http://www.pactmart.com/>
> >> > > > > www.pactmart.com | Freelancing made easy.
> >> > > > > GitHub <https://github.com/ebenezergraham> | LinkedIn
> >> > > > > <https://www.linkedin.com/in/ebenezer-graham/> | Twitter
> >> > > > > <https://twitter.com/pactmart> | Facebook
> >> > > > > <https://www.facebook.com/pactmart> | Website
> >> > > > > <http://ebenezergraham.github.io/>
> >> > > > >
> >> > > > > *skype*:
> >> > > > > ebenezer.graham
> >> > > > > | *Phone*:
> >> > > > > +230 5840 9206 <+230%205840%209206>
> >> > > > >
> >> > > > >
> >> > > > > *“Practice isn’t the thing you do once you’re good. It’s the
> >> thing you
> >> > > do
> >> > > > > that makes you good.”*
> >> > > > > -*Malcolm Gladwell*
> >> > > > >
> >> > > > >
> >> > > > >
> >> > > >
> >> > >
> >>
> >
> >
>

Reply via email to