-
Notifications
You must be signed in to change notification settings - Fork 34
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
Creating Contribution Guide #644
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Trying to fix the the issues from another branch via creating a new branch Co-Authored-By: dorian451 <112524240+dorian451@users.noreply.github.com>
DarknessUnder
added
documentation
Improvements or additions to documentation
Priority 1
Critical Issue
labels
Jan 30, 2023
This merge is a test to see if Lighthouse CI is being honest to us or not |
skyjune0824
added a commit
that referenced
this pull request
Apr 7, 2023
* Creating Contribution Guide (#644) This merge is a test to see if Lighthouse CI is being honest to us or not Co-authored-by: dorian451 <112524240+dorian451@users.noreply.github.com> * Fix docker (#655) * Update requirements.txt * Update requirements.txt * Color Mode Button/Notification -Changed the notification button to a switch -Added brief text in front so user knows function - Moved toast notification to top-center of screen * Delete workspace.xml * Dropdown added drop down to condense all 3 functions of button/moon * Drop Down Funcitonal Drop down logic is now functional, still need to add more styling * Updated Notifications and functions Changed the text and color of notifications. added some logic for functions. currently have one bug where sometimes the first mode clicked is the opposite. * Navbar highlight current page (#632) * add highlight to currently selected page in navbar (this took a lot longer than it should have) * make border thicker * fix navbar only being highlighted on top-level pages * beep boop Github Action Master Branch CI Prettier hard at work * Font and color branch (#625) * Update CourseExplorer.vue Changed the color of the title same ad other pages. * Font_and_color_changed * beep boop Github Action Master Branch CI Prettier hard at work * See if I can't specify sqlAlchemy This is a bad news. Since the localhost Docker appears to require us to specify the version. * remove 2020 data * Updated Toggle Style Logic Believe to have resolved bug, now working as intended. Need to change text color on dark mode so that its more readable * More updates comments to understand logic and other updates * Cleaning up just cleaning up code and testings * New logic update updated function logic to hopefully fix errors * Fixed Dropdown Colors Dark mode hover color changed so that the text is more visible to the user * update to running fall 2023 instead pf spring 2023 * change to get summer class info * fixed a summer address link * Adding Current Item Highlight trying to change background color based on currently selected item * changes explore icon (#661) changes explore icon * beep boop Github Action Master Branch CI Prettier hard at work * Update CourseListing.vue (#667) * beep boop Github Action Master Branch CI Prettier hard at work * Current Mode Selected Feature Update Updated some logic and testing, not fully complete * More updates Still working out logic, changed color to what will most likely be close to final color * Clean up removing unnecessary code * Bump json5 from 2.1.3 to 2.2.3 in /tests/integration (#638) Bumps [json5](https://github.com/json5/json5) from 2.1.3 to 2.2.3. - [Release notes](https://github.com/json5/json5/releases) - [Changelog](https://github.com/json5/json5/blob/main/CHANGELOG.md) - [Commits](json5/json5@v2.1.3...v2.2.3) --- updated-dependencies: - dependency-name: json5 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Working on fixing cookies reworking logic to fix cookie issue not following device theme * Revert "Working on fixing cookies" This reverts commit a27eab5. * Update cookie trying to fix logic to update cookies correctly * Fixed Cookie believed to have fixed cookie issue, atleast it works on my local device * Updating Default Logic changed default logic to toggle follow theme if user gets rid of cookie or its first time loading site * beep boop Github Action Master Branch CI Prettier hard at work * Update ci.yaml --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Harry Tan <32246618+DarknessUnder@users.noreply.github.com> Co-authored-by: dorian451 <112524240+dorian451@users.noreply.github.com> Co-authored-by: Troy-bailan-King <97757033+Troy-bailan-King@users.noreply.github.com> Co-authored-by: Matt Lammon <mlammon11@gmail.com> Co-authored-by: Matt Lammon <91478985+mlammon11@users.noreply.github.com> Co-authored-by: I'm a Github Action <beepboop@github.com> Co-authored-by: charleschae12 <112535899+charleschae12@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
jonha1
pushed a commit
that referenced
this pull request
Apr 21, 2023
* Update NewCourseScheduler.vue Made the hide and open button for the side menu * Update NewCourseScheduler.vue * Update NewCourseScheduler.vue Added the close sidebar function and after the sidebar is closed the schedule image expended to the full screen. Also after the sidebar is closed new button called "Open" appears and if we click it the sidebar opens again. * Update NewCourseScheduler.vue * Update NewCourseScheduler.vue * Update NewCourseScheduler.vue * Update NewCourseScheduler.vue * Sidebar creation * Update Sidebar.vue * Update NewCourseScheduler.vue * Update NewCourseScheduler.vue * Button Activate * Fix * Update NewCourseScheduler.vue * Syntax error fixed * Animation prototype * Finally functional animation * Update NewCourseScheduler.vue * Size and padding fix * Column dynamics * Update NewCourseScheduler.vue * Merge * Merge complete Please work * Update NewCourseScheduler.vue * Finalize * Collapse button design * Animation finalize * [CodeFactor] Apply fixes * Unclickable problem solved * Color-fix for darkmode * Bug fix font-size and color was re-defined, so I fixed it. * Update NewCourseScheduler.vue Defalt status of sidebar is now 'open' * Finalize fixation & animations Animatino is now rotating for only Y-axis * Merge from master to my branch (#680) * Creating Contribution Guide (#644) This merge is a test to see if Lighthouse CI is being honest to us or not Co-authored-by: dorian451 <112524240+dorian451@users.noreply.github.com> * Fix docker (#655) * Update requirements.txt * Update requirements.txt * Color Mode Button/Notification -Changed the notification button to a switch -Added brief text in front so user knows function - Moved toast notification to top-center of screen * Delete workspace.xml * Dropdown added drop down to condense all 3 functions of button/moon * Drop Down Funcitonal Drop down logic is now functional, still need to add more styling * Updated Notifications and functions Changed the text and color of notifications. added some logic for functions. currently have one bug where sometimes the first mode clicked is the opposite. * Navbar highlight current page (#632) * add highlight to currently selected page in navbar (this took a lot longer than it should have) * make border thicker * fix navbar only being highlighted on top-level pages * beep boop Github Action Master Branch CI Prettier hard at work * Font and color branch (#625) * Update CourseExplorer.vue Changed the color of the title same ad other pages. * Font_and_color_changed * beep boop Github Action Master Branch CI Prettier hard at work * See if I can't specify sqlAlchemy This is a bad news. Since the localhost Docker appears to require us to specify the version. * remove 2020 data * Updated Toggle Style Logic Believe to have resolved bug, now working as intended. Need to change text color on dark mode so that its more readable * More updates comments to understand logic and other updates * Cleaning up just cleaning up code and testings * New logic update updated function logic to hopefully fix errors * Fixed Dropdown Colors Dark mode hover color changed so that the text is more visible to the user * update to running fall 2023 instead pf spring 2023 * change to get summer class info * fixed a summer address link * Adding Current Item Highlight trying to change background color based on currently selected item * changes explore icon (#661) changes explore icon * beep boop Github Action Master Branch CI Prettier hard at work * Update CourseListing.vue (#667) * beep boop Github Action Master Branch CI Prettier hard at work * Current Mode Selected Feature Update Updated some logic and testing, not fully complete * More updates Still working out logic, changed color to what will most likely be close to final color * Clean up removing unnecessary code * Bump json5 from 2.1.3 to 2.2.3 in /tests/integration (#638) Bumps [json5](https://github.com/json5/json5) from 2.1.3 to 2.2.3. - [Release notes](https://github.com/json5/json5/releases) - [Changelog](https://github.com/json5/json5/blob/main/CHANGELOG.md) - [Commits](json5/json5@v2.1.3...v2.2.3) --- updated-dependencies: - dependency-name: json5 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Working on fixing cookies reworking logic to fix cookie issue not following device theme * Revert "Working on fixing cookies" This reverts commit a27eab5. * Update cookie trying to fix logic to update cookies correctly * Fixed Cookie believed to have fixed cookie issue, atleast it works on my local device * Updating Default Logic changed default logic to toggle follow theme if user gets rid of cookie or its first time loading site * beep boop Github Action Master Branch CI Prettier hard at work * Update ci.yaml --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Harry Tan <32246618+DarknessUnder@users.noreply.github.com> Co-authored-by: dorian451 <112524240+dorian451@users.noreply.github.com> Co-authored-by: Troy-bailan-King <97757033+Troy-bailan-King@users.noreply.github.com> Co-authored-by: Matt Lammon <mlammon11@gmail.com> Co-authored-by: Matt Lammon <91478985+mlammon11@users.noreply.github.com> Co-authored-by: I'm a Github Action <beepboop@github.com> Co-authored-by: charleschae12 <112535899+charleschae12@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Update NewCourseScheduler.vue * Revert "Merge from master to my branch (#680)" This reverts commit 3f49f69. --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Charles Chae <charleschae12@gmail.com> Co-authored-by: codefactor-io <support@codefactor.io> Co-authored-by: Harry Tan <32246618+DarknessUnder@users.noreply.github.com> Co-authored-by: dorian451 <112524240+dorian451@users.noreply.github.com> Co-authored-by: Troy-bailan-King <97757033+Troy-bailan-King@users.noreply.github.com> Co-authored-by: Matt Lammon <mlammon11@gmail.com> Co-authored-by: Matt Lammon <91478985+mlammon11@users.noreply.github.com> Co-authored-by: I'm a Github Action <beepboop@github.com> Co-authored-by: charleschae12 <112535899+charleschae12@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Trying to fix the issues from another branch via creating a new branch
Co-Authored-By: dorian451 112524240+dorian451@users.noreply.github.com
Issue
Some CI checks does not work for the original guide
Example: