RepoSense
RepoSense copied to clipboard
[#1665] rename frontend component
Add changes according to [#1665]
Proposed commit message
The components are all named with a prefix -v
; however, most reserved syntax used by Vue uses the -v
.
Let's change the -v
prefix to prefix -c
, so that it will be less likely to cause confusion
Other information
I tried to just remove the prefix previously, but the summary
is a reserved word in html, thus the v-summary
can't be renamed. So I add -c
as a prefix.
Hi @Zhou-Jiahao-1998 , I have solved the conflict and now created a new pull request
@LuoZhijie-tom Hi there, please update the cypress tests as well to reflect the changes. You can find the files under frontend/crypress
folder.
@LuoZhijie-tom Hi there, please update the cypress tests as well to reflect the changes. You can find the files under
frontend/crypress
folder.
Hi @Zhou-Jiahao-1998 , I have modified the code and it passed the cypress test now, could you have a look?
Guys, shall we push this PR forward?
@dcshzj I have updated the diagrams, could you have a look?
Guys, shall we merge this?
The following links are for previewing this pull request:
- Dashboard Preview: https://dashboard-1836-pr-reposense-reposense.surge.sh
- Docs Preview: https://docs-1836-pr-reposense-reposense.surge.sh