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

Test #1081

Closed
wants to merge 199 commits into from
Closed

Test #1081

wants to merge 199 commits into from

Conversation

softcat477
Copy link
Contributor

Resolves: (#ID-of-the-issue)

  • I passed the docker hub test, and images can be built successfully.
  • I passed the GitHub CI test, so rodan functionalities and jobs work.

(Describe the changes you've made and the purpose of this PR)

jackyyzhang03 and others added 30 commits May 25, 2023 09:11
Removed some unnecessary divs, and added some css selectors to certain elements in order to be able to add css styles.
Dependencies still need to be removed from package.json. I will do it later once this decision is finalized.
Mostly adding width and height attributes to ensure application takes up the full browser page.
Some HTML reformatting + a bunch of CSS updates (mostly related to the main Rodan navbar).
- Refactor code to support new design of the navbar navigation tree.
- Update CSS styles
- Clean up HTML
- Refactored many of the classnames used within Rodan-client. They are now more general and consistent across the application, which allowed me to remove a lot of CSS that was being applied to specific elements rather than reusable classnames.
- Overhauled the table filters. They are now always present and don't have to be selected from a dropdown in order to be visible.
- Added a lot of styles to improve the UI. Mostly related to tables and table filters.
add base.py which has the rodan task object for this job
sabrina0822 and others added 29 commits August 24, 2023 12:17
- By pushing a dummy image to another ddmal's docker repo
Trigger webhook and send message to Slack
Fix typo in sending workflow failure emails
@softcat477 softcat477 closed this Oct 4, 2023
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.

9 participants