Replace the json files with google sheets

Did you try adding images with different sizes?

1 Like

Yeah, I did it @anjisvj . We can resize the image manually or we can use a template that was provided by the site.

1 Like

Awsome the reason I suggested cloudinary is we can use their APIs specially for resize. So we can skip the manually resizing part.

1 Like

Yeah! I saw that too. So what is the next step should I update the (https://github.com/sef-global/sef-site/pull/985) PR with the Cloudinary service or is there any improvement to do @anjisvj?

1 Like

Let’s wait for the confirmation. @jaye WDYT?

1 Like

@EngTeam I need a Cloudinary account created by the sef email to upload the images. So can you create a Cloudinary account and give me the account password then I can upload the images to that and add the URL’s to the google sheet :grin:

1 Like

Yeah @Gajindu_Bandara @anjisvj I think Cloudinary has all the features we need. It fits our requirement let’s move forward with this.

1 Like

I replaced the JSON files in the Onelive and the IRC pages with google sheets and still, there are some remaining pages (ScholarX and Team page). I’ll create issues for them. :grin:

Currently, I’m working on the SEF Cloudinary account to create folders and to upload the images in order so, after that, I’ll create some issues to replace the images with the Cloudinary URLs as well. :hugs:

@EngTeam

2 Likes

@Gajindu_Bandara I saw there’s a cloudenery’s url prefix on mustache templates.

If getting the direct url is easier than getting the image tag from the cloudenery dashboard, I recommend you to use add direct links to the sheets instead of image tags.

However, if it supports some resize options by providing dimensions from the url, then the current implementation is totally fine.

I went through their documentation. There’s a way to resize or crop images by providing url params. Therefore, the current implementation is better. Good work @Gajindu_Bandara and @Akeel_malik !

Please add the exact dimensions in the mustache templates, so that we can increase the loading time.

Applied to:

Also,
@Gajindu_Bandara and @Akeel_malik please link the pr links to relevant hive threads next time onwards. It will be a great help when tracking the tasks. :hugs:

Good job as always!

1 Like

@jaye we will look into it and make the requested changes asap

Have we lost the names in the process :

Yeah ok I’ll update the PR

I guess something went wrong I’ll check it!

Fixed the issue. There was a mistake in the google sheet and I fixed it Now it’s working!


@Gajindu_Bandara The images of the IRC page has also gone missing. Will you be able to have a look at that too?

The problem is that the image column of the IRC researcher’s google sheet is updated with the Cloudinary links but the PR for Changing image URLs for the IRC page is still unmerged

@jaye I used the scale parameter to scale the images to the required size. Is this ok or is there anything to be changed :thinking:

Perfect!! Merged the PR! Thanks for getting this done, Gajindu!! :hugs:

IRC:


We need a QA engineer :stuck_out_tongue:

1 Like