search

novuhq / novu-java

Java SDK for Novu - The open-source notification infrastructure for engineers. 🚀
MIT License
26 stars 22 forks source link

Refactor Notifications API to conform to new design #52

Closed mayorJAY closed 11 months ago

mayorJAY commented 11 months ago

We want to refactor the Notifications API to address compatibility issues with Micronaut and Quarkus arising from the library's reliance on Spring framework.

We would like to have the Events API migrated to the new REST dependency and ensure the on going updates to the unit tests to validate the modifications and uphold code quality.

For guidance on the expected changes, kindly refer to this pull request #54.

PLEASE BRANCH OFF STAGING BRANCH AND ALL PRS SHOULD GO TO STAGING BRANCH FOR THESE CHANGES

debanjanc01 commented 11 months ago

@mayorJAY is this up for grabs?

mayorJAY commented 11 months ago

Yes, let me know if you want it assigned to you

debanjanc01 commented 11 months ago

Yes, let me know if you want it assigned to you

Yes, thanks!

mayorJAY commented 11 months ago

@debanjanc01 there seems to be no traction on this issue. Are you still working on it?

sccalabr commented 11 months ago

I'll pick this up if there is no traction

debanjanc01 commented 11 months ago

Hey @mayorJAY , yes I'm working on it.

debanjanc01 commented 11 months ago

Hey @mayorJAY , The PR has been raised.