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

Blob types not saved #129

Closed
ghost opened this issue Apr 25, 2020 · 7 comments · Fixed by #162
Closed

Blob types not saved #129

ghost opened this issue Apr 25, 2020 · 7 comments · Fixed by #162
Labels
bug Something isn't working $$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ v1.2.0 1.2.0 Release label $200 https://www.jhipster.tech/bug-bounties/
Milestone

Comments

@ghost
Copy link

ghost commented Apr 25, 2020

Blob images file are not saved in sqlite.

@ghost ghost added the bug Something isn't working label Apr 25, 2020
@ghost
Copy link
Author

ghost commented Aug 5, 2020

Today ImageBlob jdl type instead is saved

@mraible mraible added $$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ $200 https://www.jhipster.tech/bug-bounties/ labels Aug 20, 2020
@ghost ghost self-assigned this Aug 21, 2020
@ghost ghost added this to To do in nodejs blueprint via automation Aug 21, 2020
@ghost
Copy link
Author

ghost commented Aug 21, 2020

Hi @mraible, I have tested the creation of the blob with the test-integration jdl for angular and react.

entity DemetraUser {
	gender Gender required,
	address String required,
	borndate LocalDate required,
	cf String required unique,
	usertype UserType required,
	profileimage ImageBlob
}

entity SingleCourse {
	title String required unique,
	description String,
	isnotonlyfordegree Boolean required,
	cfu Integer,
	courseimage Blob,
	amount BigDecimal
}

So we have an profileimage ImageBlob type in DemetraUser and a courseimage Blob type in SingleCourse.

The bug does not exist now, maybe for the latest development.

demetra-user-angular-1
demetra-user-angular-2
demetra-user-react-1
demetra-user-react-2

I have only noted that for the angular client the blob must be small (almost 4-5 MB), otherwise there is error for the request too large.

Then, I have seen that we have to add the nullable value attribute in the backend entity if there isn't required from jdl.
For example:

@Column({ type: 'boolean', name: 'isnotonlyfordegree', nullable: true })

So I will do this fix.

@ghost ghost added this to the 1.2.0 milestone Aug 21, 2020
@ghost ghost added the v1.2.0 1.2.0 Release label label Aug 21, 2020
ghost pushed a commit that referenced this issue Aug 21, 2020
@ghost ghost moved this from To do to In progress in nodejs blueprint Aug 21, 2020
ghost pushed a commit that referenced this issue Aug 21, 2020
ghost pushed a commit that referenced this issue Aug 21, 2020
ghost pushed a commit that referenced this issue Aug 21, 2020
ghost pushed a commit that referenced this issue Aug 21, 2020
@ghost ghost linked a pull request Aug 21, 2020 that will close this issue
@ghost ghost closed this as completed in #162 Aug 21, 2020
nodejs blueprint automation moved this from In progress to Done Aug 21, 2020
ghost pushed a commit that referenced this issue Aug 21, 2020
@ghost ghost reopened this Aug 21, 2020
nodejs blueprint automation moved this from Done to In progress Aug 21, 2020
@ghost
Copy link
Author

ghost commented Aug 21, 2020

@mraible now you can try and check it ;)

@ghost ghost moved this from In progress to Review in progress in nodejs blueprint Aug 21, 2020
@mraible
Copy link
Contributor

mraible commented Aug 31, 2020

@amanganiello90 Is this available in a released version of this blueprint?

@ghost
Copy link
Author

ghost commented Sep 1, 2020

I don't know if the last release is ok, I have tested the latest commits

@ghost
Copy link
Author

ghost commented Sep 3, 2020

I have just released the new 1.2.0 version! You can check it!

@ghost
Copy link
Author

ghost commented Sep 3, 2020

@ghost ghost closed this as completed Sep 3, 2020
nodejs blueprint automation moved this from Review in progress to Done Sep 3, 2020
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working $$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ v1.2.0 1.2.0 Release label $200 https://www.jhipster.tech/bug-bounties/
Projects
Development

Successfully merging a pull request may close this issue.

1 participant