-
Notifications
You must be signed in to change notification settings - Fork 46
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
Add Program overview to full-time-program.html #455
base: develop
Are you sure you want to change the base?
Conversation
Added the ordered list from the sponsorships page to the full time program page, where the intended audience are more likely to see it
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
I didn't make a PR for that branch yet because there were other changes intended to be on that branch from our Consulting page discussion. If you want it in now, you can, but I was planning to do this aligned to the launch of the new Consulting page because some info is going to move to that page. I also wanted to avoid linking to the middle of the sponsorships page, when instead I'd like to wait to link to the fully-consulting-focused page to avoid prospective-participant confusion about which parts of the page are meant for them. This list is coming in from the Sponsor page, but it's not actually intended to be ready yet. Could we maybe convert to a draft? I didn't intend for that commit to be merged as it is. |
@MaggieFero I am definitely okay with having a shiny new consulting page here. I saw the random commit and didn't want it to get swept under the rug! Moving to draft ✏️ |
Thank you! I had a note to finish the branch as one of the Consulting Page google doc action items, but I didn't have a clear enough set of objectives for that changeset yet to create even a draft PR. |
…se this has taken a while
templates/full-time-program.html
Outdated
<li>Full-Time Software Engineering Program (Technical Training)</li> | ||
<li>Seeker Program (Job Search Support for Graduates of Full-Time Program)</li> | ||
<li> | ||
<a href="https://techtonica.org/sponsor/#techtonica-consulting" alt="hyperlink going to sponsorships webpage Techtonica Consulting section">Techtonica Consulting </a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@daaimah123 Would you mind if I removed the hyperlink here and we instead add links to this, the Seeker FAQ, and the full-time program description below in a new PR? We could do it after this is merged, too, to keep this around the one content move
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Go for it!
It looks like this was moved from non-webpage content, so I think it's ready to review and the only outstanding thing is that I'm nervous about linking to a thing we're about to remove, so I'd like to wait on that if it's okay with @daaimah123 . Otherwise, this is in a state where you can merge at will! |
@MaggieFero can you clarify which is the thing we are linking to that will be removed? Do you mean replace hyperlink with the consulting page link? |
@daaimah123 The current hyperlink is to a page section that's deleted in the same PR that introduces the consulting page. I think the consulting page has been merged, in which case we could just update the hyperlink destination and we'd no longer be linking to imminently-deleted content. |
There were proposed changes to the FT page providing additional context about programming offered, that were committed but not given a PR for consideration.
There is presently no mention of the seeker program or the consulting program on the /full-time-program page. There is mention of Techtonica Consulting on the /sponsors page, but no mention of the seeker program
The ordered list from the sponsorships page to the full time program page, where the intended audience are more likely to see it.