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

Add redirect for broken links on "Project Team Meetings" page #4906

Closed
3 tasks done
Tracked by #4868
t-will-gillis opened this issue Jul 2, 2023 · 13 comments · Fixed by #5106
Closed
3 tasks done
Tracked by #4868

Add redirect for broken links on "Project Team Meetings" page #4906

t-will-gillis opened this issue Jul 2, 2023 · 13 comments · Fixed by #5106
Assignees
Labels
Complexity: Small Take this type of issues after the successful merge of your second good first issue P-Feature: Events https://www.hackforla.org/events/ P-Feature: Project Meetings https://www.hackforla.org/project-meetings role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.25pt Can be done in 0.5 to 1.5 hours time sensitive Needs to be worked on by a particular timeframe

Comments

@t-will-gillis
Copy link
Member

t-will-gillis commented Jul 2, 2023

Overview

On our project-meetings and events pages the hrefs to "People Depot" and "Tables" link to an undefined page "https://www.hackforla.org/projects/people-depot". We want the href for "People Depot" and "Tables" to link to each project's GitHub account until they have pages in the _projects folder.

Action Items

The API_ENDPOINT for data on the project meetings page comes from VRMS. The query to VRMS returns an "event" object, and event.hflaWebsiteUrl == "" for both "People Depot" and "Tables" since neither has a project page on the website.

  • In the file api-events.js we want to redirect the links for "People Depot" and for "Tables" by inserting the following code after if (event) { on Line 35 and before let eventHtml; on Line 36:
// If a project doesn't have an hflaWebsiteUrl, redirect to the project's GitHub page
if (event.hflaWebsiteUrl == "") {
  event.hflaWebsiteUrl = event.githubUrl
}
  • Near the end of the same file in function display_object(item), we need to query the githubUrl from VRMS by adding the following entry in const rv_object after the hflaWebsiteUrl:
githubUrl: item.project.githubUrl,

Resources/Instructions

Screenshot showing current link when "People Depot" is selected on "Project Meetings" page

peopledepot1b

Screenshot showing current link when "People Depot" is selected on "Events" page

peopledepot2

@github-actions github-actions bot added Feature Missing This label means that the issue needs to be linked to a precise feature label. role missing labels Jul 2, 2023
@github-actions

This comment was marked as outdated.

@t-will-gillis t-will-gillis changed the title Make a new issue Add redirect for broken links on "Project Team Meetings" page Jul 2, 2023
@t-will-gillis t-will-gillis added role: front end Tasks for front end developers role: back end/devOps Tasks for back-end developers time sensitive Needs to be worked on by a particular timeframe Complexity: Small Take this type of issues after the successful merge of your second good first issue P-Feature: Project Meetings https://www.hackforla.org/project-meetings size: 0.25pt Can be done in 0.5 to 1.5 hours Ready for Prioritization and removed Feature Missing This label means that the issue needs to be linked to a precise feature label. role missing labels Jul 2, 2023
@ExperimentsInHonesty

This comment was marked as outdated.

@ExperimentsInHonesty ExperimentsInHonesty added ready for dev lead Issues that tech leads or merge team members need to follow up on and removed Ready for Prioritization labels Jul 2, 2023
@t-will-gillis t-will-gillis added P-Feature: Events https://www.hackforla.org/events/ Ready for Prioritization and removed ready for dev lead Issues that tech leads or merge team members need to follow up on labels Jul 2, 2023
@ExperimentsInHonesty

This comment was marked as outdated.

@ExperimentsInHonesty ExperimentsInHonesty added ready for dev lead Issues that tech leads or merge team members need to follow up on and removed Ready for Prioritization labels Jul 4, 2023
@t-will-gillis

This comment was marked as outdated.

@t-will-gillis t-will-gillis added Ready for Prioritization and removed ready for dev lead Issues that tech leads or merge team members need to follow up on labels Jul 4, 2023
@t-will-gillis

This comment was marked as resolved.

@ExperimentsInHonesty

This comment was marked as resolved.

@github-actions
Copy link

Hi @yffu, thank you for taking up this issue! Hfla appreciates you :)

Do let fellow developers know about your:-
i. Availability: (When are you available to work on the issue/answer questions other programmers might have about your issue?)
ii. ETA: (When do you expect this issue to be completed?)

You're awesome!

P.S. - You may not take up another issue until this issue gets merged (or closed). Thanks again :)

@yffu
Copy link
Member

yffu commented Jul 27, 2023

i. Availability: After 5 PT Daily
ii. ETA: 07/29 (Sat)

@yffu
Copy link
Member

yffu commented Jul 29, 2023

Have a blocker/question, wondering if someone could help:

  1. Blockers:
    on project-meetings page there's an underscore after the project name
    add-redirect-for-broken-links-4906_0

on events page there is "All Teams"
add-redirect-for-broken-links-4906_2

working on the 'add-redirect-for-broken-links-4906' branch on my forked repo.

I was thinking to change the project-meetings underscore to match the events page "All Team" but wanted to check. Thanks in advance for your help!

@yffu yffu added the Status: Help Wanted Internal assistance is required to make progress label Jul 29, 2023
@t-will-gillis
Copy link
Member Author

t-will-gillis commented Jul 29, 2023

Hey @yffu - Good question- thanks for asking!

The short answer is that this needs to be changed in VRMS, which we don't have access to. I will ask Matt P. or Bonnie if they can edit this.

Here's the long answer: project managers for each project enter details about their meetings into the VRMS app. The "Project Meetings" (and "Events") pages are querying data from VRMS- see api-events.js. Here is the object data that the page receives from VRMS:

See lower left: People Depot and Community of Practice: Engineering

Screenshot 2023-07-29 094119

So the underscore on the "Project Meetings" page is coming from the entry for "dsc" in People Depot's VMRS.

@t-will-gillis
Copy link
Member Author

@MattPereira could you edit People Depot's VMRS entry for "dsc" in their Thursday 4-5 meeting? Currently it there is an underscore, so could this be changed to a blank "" or maybe "Weekly meeting" ? Last I heard, no one at People Depot has access to VMRS...

@MattPereira
Copy link
Contributor

MattPereira commented Jul 29, 2023

@t-will-gillis got u fam its a blank string now ""

also i'll reach out to fang to ensure he has access

@yffu
Copy link
Member

yffu commented Jul 30, 2023

Thanks for the answer @t-will-gillis. It makes sense that it's from vrms so that it's flexible. Much appreciate the explanations and the examples in the screenshots.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Small Take this type of issues after the successful merge of your second good first issue P-Feature: Events https://www.hackforla.org/events/ P-Feature: Project Meetings https://www.hackforla.org/project-meetings role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.25pt Can be done in 0.5 to 1.5 hours time sensitive Needs to be worked on by a particular timeframe
Projects
Development

Successfully merging a pull request may close this issue.

5 participants