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

upstream #10

Merged
merged 47 commits into from
Jun 14, 2024
Merged

upstream #10

merged 47 commits into from
Jun 14, 2024

Conversation

shivam1608
Copy link
Owner

No description provided.

shivam1608 and others added 30 commits April 22, 2022 20:54
Domains.getRandomDomain() returns an Domain object.
It does not override toString(), so
"thisismychoice@"+Domains.getRandomDomain() will be like
thisismychoice@me.shivzee.util.Domain@7fb4f2a9
This cause following Exception :
javax.security.auth.login.LoginException: Something went wrong while
creating account! Try Againjavax.security.auth.login.LoginException:
Account Already Exists! Error 422
	at me.shivzee.util.JMailBuilder.createAndLogin(JMailBuilder.java:94)

To fix this, I called getDomainName() to get real domain name, and it
worked fine. :)
Sorry, since I'm in army, all  I can use in here is just Eclipse
Portable XD
Added Reusability of Code & Updated Java Docs
shivam1608 and others added 17 commits November 6, 2022 15:32
2. Added created/updated time in ZonedDateTime format
* Added if condition for hasAttachments

* As per review comments, modified the if block to minimal

---------

Co-authored-by: aarahman <aarahman@tekion.com>
* Added if condition for hasAttachments

* Added safeEval() so that if any field is null calling toString will not cause issue

* Changed the primitive data type to Wrapper type in POJO classes

---------

Co-authored-by: aarahman <aarahman@tekion.com>
@shivam1608 shivam1608 marked this pull request as ready for review June 14, 2024 16:09
@shivam1608 shivam1608 merged commit 62c8030 into development Jun 14, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants