website icon indicating copy to clipboard operation
website copied to clipboard

Refactor JavaScript so that Project Meetings page sources meeting data from vrms_data.json

Open roslynwythe opened this issue 1 year ago • 1 comments

Dependency

  • [x] #6023
  • [ ] #6369

Overview

As developers we need to refactor the JavaScript behind the Project Meetings page, so that it retrieves meeting data from the file data/external/vrms_data.json, in order to reduce the latency and improve the reliability of the webpage.

Action Items

  • [ ] For reference, view the code in assets/js/right-col-content.js and understand how the event data is retrieved from assets/js/utility/vrms-events.js
  • [ ] Refactor the code in assets/js/project-meetings.js to source data from data/external/vrms_data.json, using function(s) in assets/js/utility/vrms-events.js and perform testing to confirm that functionality on the Project Meetings webpage [^1] is unchanged.
  • [ ] Note that events from all projects should be displayed, except for events with the project name "test" or "Hack4LA".
  • [ ] Open the file assets/js/right-col-content.js and refactor the code to reference the function(s) in assets/js/utility/vrms-events.js and perform testing to confirm that functionality on the Project pages[^2] is unchanged.

Resources/Instructions

[^1]:Project Meetings webpage

roslynwythe avatar Jan 04 '24 09:01 roslynwythe

Hi @roslynwythe.

Please don't forget to add the proper labels to this issue. Currently, the labels for the following are missing: Complexity, Role, Feature

NOTE: Please ignore the adding proper labels comment if you do not have 'write' access to this directory.

To add a label, take a look at Github's documentation here.

Also, don't forget to remove the "missing labels" afterwards. To remove a label, the process is similar to adding a label, but you select a currently added label to remove it.

After the proper labels are added, the merge team will review the issue and add a "Ready for Prioritization" label once it is ready for prioritization.

Additional Resources:

github-actions[bot] avatar Jan 04 '24 09:01 github-actions[bot]

Hello @buneeIsSlo, we appreciate you taking on issue #6059, however it looks like you've already worked on enough issues of this complexity. Try checking out some of the issues of the next complexity from the Prioritized Backlog :)

We are going to unassign you from this issue so you can take on another issue.

Hfla appreciates you! :)

P.S. There is one exception to this rule/automation, and that is if you were away for a long time, and need to do the issue ladder again. If that is the case, please post the following note on the issue and on your Skills Issue (Pre-work Checklist). A Merge team member will reassign you to this issue, and will help you get assigned to subsequent issues up to medium size.

I am returning after a significant time away, and need to do the issue ladder again. Please assign me back to this issue.

HackforLABot avatar Oct 03 '24 06:10 HackforLABot