Skip to content

Add flow#24

Open
corincewang wants to merge 31 commits into
mainfrom
main_add_flow
Open

Add flow#24
corincewang wants to merge 31 commits into
mainfrom
main_add_flow

Conversation

@corincewang

Copy link
Copy Markdown
Contributor

Locally passed lint, test, and flow. Trial to see whether works on github check.

actions-user and others added 30 commits January 29, 2024 17:17
Changes still need to be tested. There may be some issues with the interaction between post importance and reverse order sorting.
Setter: posts.mark_important, posts.mark_unimportant (called via postAPI.important and postsAPI.unimportant respectively)
Getter: posts.is_important
Added sorting posts by priority
Add getters, setters, and test cases for importance attribute
added two sections in template files, but wait for backend to add pin…
Add the button for instructors to pin post
Change background color when pinned state is changed
…rk). For this one, we have merged the backend with the dropdown menu still working, but no two sections and background color
@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 8362550843

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.01%) to 85.028%

Files with Coverage Reduction New Missed Lines %
src/groups/join.js 1 92.05%
Totals Coverage Status
Change from base Build 8353068226: -0.01%
Covered Lines: 22650
Relevant Lines: 25399

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants