Skip to content

Feature/admin create event #40

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.

Sign up for GitHub

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

Merged
robertlb4 merged 19 commits into from
Apr 14, 2020
Merged

Feature/admin create event #40

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
3468bb3
login and register created
rita7elyz Mar 9, 2020
52a4d72
login and register created
rita7elyz Mar 9, 2020
f670394
merge origin/dev
rita7elyz Mar 10, 2020
24d5e29
Merge branch 'dev' of https://github.com/SoftStackFactory/my-request-…
rita7elyz Mar 24, 2020
e0d9362
stepper added with form field for event name
rita7elyz Mar 26, 2020
21c70ce
added create event and manage event, slider working, tabs working
rita7elyz Mar 29, 2020
c2f7ea9
added second slider
rita7elyz Mar 29, 2020
5a1e809
added styling to sliders and stepper buttons
rita7elyz Mar 29, 2020
78ce548
content page started, hard coded image
rita7elyz Mar 31, 2020
d78e712
added styling to images and began setting up drag and drop feature
rita7elyz Apr 1, 2020
98d1688
drag and drop implemented
rita7elyz Apr 1, 2020
0df23c5
removed dummy register and login components
rita7elyz Apr 2, 2020
d9cdeb3
compiling and login and register removed
rita7elyz Apr 2, 2020
7176c68
adjusted dragdrop feature
rita7elyz Apr 2, 2020
fe05a17
add
rita7elyz Apr 5, 2020
0ab2bc8
started implementing two list drag and drop
rita7elyz Apr 5, 2020
a1b98bf
still getting splice error on drag and drop
rita7elyz Apr 5, 2020
5d683ce
drag and drop working
rita7elyz Apr 6, 2020
1bea99e
final commit
rita7elyz Apr 6, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion src/app/app-routing.module.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,13 @@
import { NgModule } from '@angular/core';
import { Routes, RouterModule } from '@angular/router';
import { AdminFormCreateEventComponent } from './components/admin-form-create-event/admin-form-create-event.component';
import { AdminFormManageEventComponent } from './components/admin-form-manage-event/admin-form-manage-event.component';


const routes: Routes = [];
const routes: Routes = [
{ path: 'new/advertisingconfig', component: AdminFormCreateEventComponent },
Copy link
Collaborator

@robertlb4 robertlb4 Apr 9, 2020

Choose a reason for hiding this comment

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

I think I've settled on a preference for: new/campaign, manage/campaign

I think that is clean and understandable

Copy link
Collaborator Author

@rita7elyz rita7elyz Apr 12, 2020

Choose a reason for hiding this comment

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

gotcha, i've updated it. Since I was able to use the tabs, this path might be better named something more general since it includes both create and manage functionalities. path: 'campaigns' Thoughts?

Copy link
Collaborator

@robertlb4 robertlb4 Apr 14, 2020 •
edited
Loading

Choose a reason for hiding this comment

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

Not a bad Idea. I like the idea of having the 2 routes available for us. In the future we can use them to decide which tab we start on when the component loads. So, if a link form another part of the site goes to manage/campaign then it will go to the same component but start on the manage tab.

I see you are changing it in the other PR, so it will be fine here for now.

{ path: 'manage/advertisingconfig', component: AdminFormManageEventComponent}
];

@NgModule({
imports: [RouterModule.forRoot(routes)],
Expand Down
Loading