Skip to content

Fix/wsdev 6669 ff footer links update - #572

Open
m-vojjala wants to merge 3 commits into
developfrom
fix/WSDEV-6669-FF-Footer-Links-Update
Open

m-vojjala wants to merge 3 commits into
developfrom
fix/WSDEV-6669-FF-Footer-Links-Update

Conversation

@m-vojjala

Copy link
Copy Markdown
Contributor

PR Checklist & Notes

What type of PR is this? (check all applicable)

  • New component
  • New feature
  • Refactor
  • Bug fix
  • Optimization / Build Tools
  • Documentation Update

Is this a breaking change?

If yes, please note which project will be impacted.

  • Yes
  • No

Have you made an entry in the Changelog?

If not, please make an entry or your PR will be declined. Add new entries under the "Unreleased" section of the changelog.

  • Yes
  • No

Have you made changes to dependencies?

If you've made changes to dependencies please note them here, adding if they've been added, removed or updated, including packages version where applicable.

  • Added
  • Updated
  • Removed
  • Combo of any of the above
  • No

QA Instructions, Screenshots, Recordings

Please test all the links of Future Funder Footer.

UI accessibility concerns?

If your PR includes UI changes, please replace this line with details on how
accessibility is impacted and tested. If there are no accessibility concerns delete this section.

@Bhoomi-Pipalia

Copy link
Copy Markdown
Contributor

@m-vojjala

The links will be from new futurefunder site not the old one.

{ name: 'Read Our Blog', href: 'https://futurefunder.carleton.ca/news' },
{ name: 'Contact Us', href: 'https://futurefunder.carleton.ca/contact' },
],
Categories: [

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@m-vojjala This will the links from the cutheme futurefunder.

EX: https://cutheme.local/futurefunder/campaign-type/arts/ will be valid for the new setup.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

This branch has not been deployed

No deployments
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.

2 participants