Conversation
Javier-varez
left a comment
Javier-varez
left a comment
There was a problem hiding this comment.
Thanks for checking that it works on your end!
|
Once you merge the PR, the webpage will get automatically updated 🚀 |
Javier-varez
left a comment
Javier-varez
left a comment
There was a problem hiding this comment.
Thanks for the changes. I left a few, mostly minor comments, but I think we should remove the extra files that are not needed for the site (all *:Zone.Identifier files and the .swp file).
Does this also show the way you'd like it to already? Or is there anything you'd like to change?
content/_index.md:Zone.Identifier
Outdated
| @@ -0,0 +1,2 @@ | |||
| [ZoneTransfer] | |||
| ZoneId=3 | |||
There was a problem hiding this comment.
What are these zone identifier files?
There was a problem hiding this comment.
I have no idea. I can check.
I think, for now its okay. In general, I would like to change the layout/add graphic but needs work, Will resume when I am back.
content/projects.md
Outdated
|
|
||
| --- | ||
|
|
||
| As part of our mission, we are actively working on projects focused on rural development, beginning with bridging the digital divide in education. These initiatives also aim to enhance local livelihoods and promote long-term community growth. Our ongoing projects are described below. |
There was a problem hiding this comment.
nit: I think the correct form is beginning by bridging instead of beginning with bridging. I think with is used if a name follows, not a verb.
There was a problem hiding this comment.
Reading again and again, i think "with" in this context is more appropriate (?) Leaving it as is now but we can discuss.
.swp
Outdated
| This is an example of a custom shortcode that you can put right into your content. You will need to add a form action to the shortcode to make it work. Check out [Formspree](https://formspree.io/) for a simple, free form service. --> | ||
|
|
||
| {{< form-contact action="https://example.com" >}} | ||
| {{< form-contact action="t4eq.community@gmail.com" >}} |
There was a problem hiding this comment.
I don't think this works as you intended. Currently it redirects to <url>/contact/t4eq.community@gmail.com
As I understand, the suggestion to use Formspree is to avoid having to write custom js code to handle the form data submission by using a free 3rd party service.
There was a problem hiding this comment.
I do not understand this very well then. I did some reading on https://github.com/formspree/blogophonic-hugo . So what do you suggest? How does it work instead?
There was a problem hiding this comment.
AFAIU we need to register with formspree and then link here whatever link we get from formspree after registering/configuring the dialog. But let's maybe handle this separately in another PR. I can have a look at it.
|
@Javier-varez The contact form is left for another PR- Made other requested changes, have a look and we could be to main in that case? |
2 participants
Added a couple of statements on vision and mission
Generating pull request to verify the setup and flow shared by Javier