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

Updating a style to be applied to the federated site. #435

Merged
merged 2 commits into from
Jan 11, 2024

Conversation

goynejennifer
Copy link
Collaborator

JLG 12012023

Updating Style in FeatureItems.scss to get the circles in the federated to work properly


JIRA Ticket: (link) (:star:)

  • Other Relevant Links (Meeting note, project page, related pull requests, etc.)

What does this Pull Request do? (:star:)

This pull request should fix the issue with the 3 circles that are on the federated preproduction site.

What's the changes? (:star:)

The FeatureItems.scss line 19 was commented out to get the three circles on the federated site working.

Example:
Updated FeatureItems.scss, line 19, commented out the height because it was messing up the circles.

How should this be tested?

To test this out push the updated .scss style file to the preproduction federated website to test if the change does work on the federated website. https://federated-pprd.dlp.cloud.lib.vt.edu/

A description of what steps someone could take to:
None

Additional Notes:

Any additional information that you think would be helpful when reviewing this PR.

Example:
This was branched off the main dev branch.

Interested parties

Tag (@ mention) interested parties

(:star:) Required fields

@goynejennifer goynejennifer assigned whunter and asthad1 and unassigned whunter Dec 1, 2023
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@goynejennifer is there a specific reason why this is set to null?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why this is set to Null did I set this to null. I probably should be set to a value.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It came from your merge request. So I guess its you.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LOL. As far as I know it's just extra files the Visual Studio creates but maybe I don't need to check in just to keep locally.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@goynejennifer I am not sure if this is required for the project. I see your local path so not sure how this is used?

Copy link

Preview branch generated at https://JLG-DevUpdateFederatedStyles.d234tdmg9icdwk.amplifyapp.com

@asthad1 asthad1 merged commit 79b8e28 into dev Jan 11, 2024
1 check passed
@goynejennifer goynejennifer deleted the JLG_DevUpdateFederatedStyles branch February 21, 2024 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants